-
-
Notifications
You must be signed in to change notification settings - Fork 523
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
fix: convert from array to dictionary #1325
fix: convert from array to dictionary #1325
Conversation
Co-authored-by: Elia Tolin <[email protected]>
c00c0f5
to
12d7b4c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's good!
We are syncing the file from here: So, to ensure that the fix isn't overwritten each sync, we'd need to adjust one layer above: |
Do you suggest then to transform the result to map above? |
@jonasbark I think it is very important to fix the problem, Stripe does not currently work on iOS. See the issue #1321 |
Yes - as in leave the source code as it is, but check for the result in the layer above. E.g. if result == [] => return [:] |
Move to #1333. |
Closing as it was moved to #1333 |
This fixes #1321 where Flutter would expect a
Map<String,dynamic>?
while iOS was sending an empty array.This resulted in
type 'List<dynamic>' is not a subtype of type 'Map<String, dynamic>?'
exception.