Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add client-side metadata charge_request_id to requests to example-ios-backend #1008

Merged
merged 2 commits into from
Aug 8, 2018

Conversation

danj-stripe
Copy link
Contributor

Summary

The sample apps are now sending a (meaningless) randomly generated UUID value through
stripe/example-ios-backend as metadata from the client.

charge_request_id: B3E611D1-5FA1-4410-9CEC-00958A5126CB

Also updating all of our links to the example-ios-backend to use the latest release
that includes support for this feature.

Motivation

IOS-356

Along with the changes from stripe/example-mobile-backend#35, this shows off some of what
metadata can be used for.

Testing

Ran each of the sample apps, created Charges and a PaymentIntent, and then manually
verified the metadata was associated with the newly created Payments in the web dashboard.

We don't have automated testing of the sample apps, and this doesn't change the SDK.

…ios-backend`

The value is meaningless, it's just a randomly generated UUID
`B3E611D1-5FA1-4410-9CEC-00958A5126CB`.

However, along with the changes from stripe/example-mobile-backend#35, this shows off some
of what `metadata` can be used for.

I am *not* adding metadata to each location where the Stripe API is directly used, only
the places that go through stripe/example-ios-backend. Many of the places that directly
deal with the stripe-ios SDK could *also* add metadata. That's being omitted for
simplicity.

See also IOS-356
Copy link
Contributor

@csabol-stripe csabol-stripe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cool

@danj-stripe danj-stripe merged commit a8922af into master Aug 8, 2018
@danj-stripe danj-stripe deleted the danj/feature/add-metadata branch August 8, 2018 23:56
csabol-stripe pushed a commit that referenced this pull request Apr 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants