firebase-ios-sdk
firebase-ios-sdk copied to clipboard
Fixed inconsistent appdistribution prompt (#8096)
Fix for issue (#8096), prompt is shown consistently.
Thanks for the contribution @a-25. A quick question, did you look into the UISceneDidActivateNotification
at all to see if it could be used instead? Seems like it could be useful and would let us avoid just looping without knowing the state.
@ryanwilson Hello. No, I didn't. Will look at it closely and return with changes or conclusions.
Hello, @ryanwilson. Implemented using UISceneDidActivateNotification. Seems to me that it is a cleaner way.
Apologies for the delay in review, will be getting to it soon! Thanks for the update as well.
A couple comments to start!
Hello, @ryanwilson. Thanks for your comments, worked them out.
Hi, @ryanwilson. Finished with comments fixing. Perhaps, I'd better resolve all conversations so that you received a notification?
Hi @a-25 - thanks for the PR!
There's a simpler fix (focused on single scenes) - does this work in your case? Or does your use case involve multiple scenes?
https://github.com/firebase/firebase-ios-sdk/pull/10288
Hi, @tejasd. My case was for multiple scenes, as I remember. At least, I cannot guarantee that the original bug appears only in one-scene apps.
Please close, merge, or comment. We plan to close stale PRs on November 28, 2023.
@ryanwilson, @tejasd: what do you think, should it be merged?
Looks like no immediate plans to move forward on this. I'll close and we can reopen if plans change.