Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

[firebase_dynamic_links] sometimes got error NSPOSIXErrorDomain Code=53 in ios #1182

Merged
merged 4 commits into from
Mar 27, 2019

Conversation

ko2ic
Copy link
Contributor

@ko2ic ko2ic commented Feb 8, 2019

As in link, there is a case of nil at the completion of handleUniversalLink.
firebase/firebase-ios-sdk#2303

Also, even if sleep, cause an error, so fixed it to tell
it to the dart side.

flutter/flutter#27810

@collinjackson
Copy link
Contributor

collinjackson commented Mar 22, 2019

I don't think our current flutter drive based integration testing can regression test this change, so I'm ok with merging it as-is. We should get rid of the NSError categories to match other plugins, but that can happen in a subsequent PR.

@collinjackson collinjackson self-requested a review March 22, 2019 11:44
@ko2ic ko2ic force-pushed the dynamiclink_sometime_nil branch from 5f0518b to a9c8f72 Compare March 25, 2019 01:21
As in link there is a case of nil at completion of handleUniversalLink.
firebase/firebase-ios-sdk#2303

Also, even if make this correction,  cause an error, so fixed it to tell
it to the dart side.
@ko2ic ko2ic force-pushed the dynamiclink_sometime_nil branch 2 times, most recently from aa687ca to 0db4f75 Compare March 25, 2019 02:42
Copy link
Contributor

@bparrishMines bparrishMines left a comment

Choose a reason for hiding this comment

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

LGTM

@ko2ic
Copy link
Contributor Author

ko2ic commented Mar 26, 2019

@collinjackson @bparrishMines

It became all green. Could you merge?

@bparrishMines
Copy link
Contributor

Yea I can merge it for you. I added a version bump and a corresponding part to the android side.

Thanks for the contribution! I'll merge once it goes green again.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bugfix cla: yes flutterfire submit queue The Flutter team is in the process of landing this PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants