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

Discrepancy between Date handling on iOS and Android #368

Closed
jonasbark opened this issue Jun 23, 2021 · 0 comments · Fixed by #392
Closed

Discrepancy between Date handling on iOS and Android #368

jonasbark opened this issue Jun 23, 2021 · 0 comments · Fixed by #392
Labels
bug Something isn't working P0 Address these issues first

Comments

@jonasbark
Copy link
Collaborator

We noticed a discrepancy between iOS / Android regarding "created" attribute on PaymentIntent on the native side:

https://github.com/stripe/stripe-react-native/blob/master/android/src/main/java/com/reactnativestripesdk/Mappers.kt#L328 -> created is a string
https://github.com/stripe/stripe-react-native/blob/master/ios/Mappers.swift#L360 -> created is an int
https://github.com/stripe/stripe-react-native/blob/master/src/types/PaymentIntents.ts#L7 -> created is a string

I'm not sure if it has an effect on React Native itself but it does on Android. Adjusting convertToUnitTimestamp on Mappers.kt on Android to return a String would fix this issue.

jonasbark added a commit to flutter-stripe/flutter_stripe that referenced this issue Jun 23, 2021
@thorsten-stripe thorsten-stripe added the bug Something isn't working label Jun 24, 2021
@thorsten-stripe thorsten-stripe added the P0 Address these issues first label Jun 25, 2021
remonh87 pushed a commit to flutter-stripe/flutter_stripe that referenced this issue Jun 27, 2021
jonasbark added a commit to flutter-stripe/flutter_stripe that referenced this issue Jul 4, 2021
* sync android code

* implement new parsing cardfield and paymentintent

* comply to new result propagation for setupintent

* sync ios

* Update dart layer

* Throw errors for easy debug + cancel login when error

* workaround for stripe/stripe-react-native#368, also adjusted canceledAt

* migrate to flutter_lints because pedantic is deprecated

* implement create token method (#165)

* implement create token method

* add create token to example app

* Add createToken for iOS

* Make example tiles scrollable

Co-authored-by: Jaime Blasco <[email protected]>

* improve documentation for Android and issues #155 #142

* fix merge issue

* add  urlscheme to initialise method

* BREAKING: rename threedsecure button config and add config theme for other buttons

* BREAKING make paymentIntentClientSecret nullable and add setupIntentClientSecret for paymentsheet

* add fontfamily and dangerouslyGetFullCardDetails to cardfield

Co-authored-by: Remon <[email protected]>
Co-authored-by: Remon Helmond <[email protected]>
Co-authored-by: Jaime Blasco <[email protected]>
softdev55 added a commit to softdev55/flutter_stripe that referenced this issue Aug 30, 2022
* sync android code

* implement new parsing cardfield and paymentintent

* comply to new result propagation for setupintent

* sync ios

* Update dart layer

* Throw errors for easy debug + cancel login when error

* workaround for stripe/stripe-react-native#368, also adjusted canceledAt

* migrate to flutter_lints because pedantic is deprecated

* implement create token method (#165)

* implement create token method

* add create token to example app

* Add createToken for iOS

* Make example tiles scrollable

Co-authored-by: Jaime Blasco <[email protected]>

* improve documentation for Android and issues #155 #142

* fix merge issue

* add  urlscheme to initialise method

* BREAKING: rename threedsecure button config and add config theme for other buttons

* BREAKING make paymentIntentClientSecret nullable and add setupIntentClientSecret for paymentsheet

* add fontfamily and dangerouslyGetFullCardDetails to cardfield

Co-authored-by: Remon <[email protected]>
Co-authored-by: Remon Helmond <[email protected]>
Co-authored-by: Jaime Blasco <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working P0 Address these issues first
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants