Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

Push stripe-android to 16.10.0 #790

Merged
merged 1 commit into from
Jun 17, 2021

Conversation

wijskinner
Copy link

Proposed changes

Here's my effort at a PR for the changes suggested in issue #785. This works for my needs in that I can continue to make payments and setup payment methods in our react-native app. Full disclosure - this is simply taking the code suggested in the issue comment #785 (comment) and making it work for my uses - I have not tested it in any other scenarios.

Types of changes

What types of changes does your code introduce to tipsi-stripe?
Put an x in the boxes that apply

  • Bugfix (a non-breaking change which fixes an issue)
  • New feature (a non-breaking change which adds functionality)
  • Possibly breaking change (fix or feature that would cause existing functionality not to work as expected)

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! Next steps are a reminder of what we are going to look at before merging your code.

  • I have added tests that prove my fix is useful or that my feature works
  • I have added necessary documentation (if appropriate)
  • I know that my PR will be merged only if it has tests and they pass

Further comments

If this is a relatively large or complex change, kick off the discussion by explaining why you choose the solution you did and what alternatives you considered.

@cybergrind cybergrind merged commit 46e72bd into tipsi:master Jun 17, 2021
@cybergrind
Copy link
Member

@wijskinner great work, thank you.

@cybergrind
Copy link
Member

I've published 9.1.0 as beta.

Folks, it would be great if you could check if it is working before public release.

@saadnaveed94 @ImPDP @designerofthing @wijskinner @ecarrera @karanmehta4797

@saadnaveed94
Copy link

Testing it now

@hbole
Copy link

hbole commented Jun 21, 2021

Is this working fine @wijskinner @saadnaveed94 @cybergrind for all kind of payments??

@hbole
Copy link

hbole commented Jun 21, 2021

tipsiStripe.setOptions({
  publishableKey: STRIPE_PUBLIC_KEY,
});

try {
        const token = await tipsiStripe.createTokenWithCard(params);
        console.log('Token', token);
        const paymentMethod = await tipsiStripe.createPaymentMethod({
          card: {
            token: token.tokenId,
          },
        });
        console.log('Payment method', paymentMethod);

        stripePaymentRequest(currency, paymentMethod.id, saveCard, null);
      } catch (e) {
        Sentry.captureException(e);
        console.log('Error here', e);
        Toast.show(
          'Error processing your payment, please try again',
          Toast.SHORT,
        );
      }

This is my code, I am trying to pay using this beta version and I get this error - Error here [Error: Keys for idempotent requests can only be used with the same parameters they were first used with. Try using a key other than 'pk_live_<complete_key>' if you meant to execute a different request.]

Please help

@hbole hbole mentioned this pull request Jun 21, 2021
@DanGDroid
Copy link

i still could not build - i have gms version conflicts

@cybergrind cybergrind linked an issue Jul 5, 2021 that may be closed by this pull request
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Google Play Update
5 participants