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

Pay plugin #4

Merged
merged 11 commits into from
Apr 6, 2021
Merged

Pay plugin #4

merged 11 commits into from
Apr 6, 2021

Conversation

jonasbark
Copy link
Member

No description provided.

Copy link
Member

@remonh87 remonh87 left a comment

Choose a reason for hiding this comment

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

for the rest looks good just check why those two files are not generated anymore

@@ -1,761 +0,0 @@
// GENERATED CODE - DO NOT MODIFY BY HAND
Copy link
Member

Choose a reason for hiding this comment

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

It is a bit suspicious that this is removed while we still use freezes in this file. Propably freeze code gen fails

Copy link
Member Author

Choose a reason for hiding this comment

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

just forgot to commit them

@@ -1,307 +0,0 @@
// GENERATED CODE - DO NOT MODIFY BY HAND
Copy link
Member

Choose a reason for hiding this comment

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

Looks like payment method codegen fails here

# Conflicts:
#	stripe_platform_interface/lib/src/models/card_field_input.dart
#	stripe_platform_interface/lib/src/models/card_field_input.freezed.dart
#	stripe_platform_interface/lib/src/models/card_field_input.g.dart
#	stripe_platform_interface/lib/src/models/payment_methods.dart
#	stripe_platform_interface/lib/src/models/payment_methods.freezed.dart
#	stripe_platform_interface/lib/src/models/payment_methods.g.dart
.setNumber(card.getString("number").orEmpty())
.build()
if (card.hasKey("token")) {
return PaymentMethodCreateParams.Card.create(card.getString("token"))
Copy link
Contributor

Choose a reason for hiding this comment

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

I had to do `PaymentMethodCreateParams.Card.create(card.getString("token")!!) otherwise Android would throw a type mismatch error. Did you not get that build error?

Copy link
Member Author

Choose a reason for hiding this comment

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

no, compiled and worked fine on the device for me with Android Studio 4.1.3

Copy link
Contributor

Choose a reason for hiding this comment

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

The React Native project also? Looks like it might be related to RN
image

Copy link
Member Author

Choose a reason for hiding this comment

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

Didn't know it was marked nullable on React Native - my bad
Can you quickly fix it? Otherwise I'll send a PR

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh yah, no worries, already done before merging: stripe/stripe-react-native@407fd89

Just wanted to flag with you in case it applied here also.

@jonasbark jonasbark merged commit b7d3962 into main Apr 6, 2021
@jonasbark jonasbark deleted the pay_plugin branch April 6, 2021 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants