firebase-ios-sdk icon indicating copy to clipboard operation
firebase-ios-sdk copied to clipboard

Fixed inconsistent appdistribution prompt (#8096)

Open a-25 opened this issue 2 years ago • 8 comments

Fix for issue (#8096), prompt is shown consistently.

a-25 avatar Jan 06 '22 10:01 a-25

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 avatar Jan 07 '22 15:01 ryanwilson

@ryanwilson Hello. No, I didn't. Will look at it closely and return with changes or conclusions.

a-25 avatar Jan 09 '22 12:01 a-25

Hello, @ryanwilson. Implemented using UISceneDidActivateNotification. Seems to me that it is a cleaner way.

a-25 avatar Jan 14 '22 15:01 a-25

Apologies for the delay in review, will be getting to it soon! Thanks for the update as well.

ryanwilson avatar Jan 21 '22 19:01 ryanwilson

A couple comments to start!

Hello, @ryanwilson. Thanks for your comments, worked them out.

a-25 avatar Jan 22 '22 11:01 a-25

Hi, @ryanwilson. Finished with comments fixing. Perhaps, I'd better resolve all conversations so that you received a notification?

a-25 avatar Feb 10 '22 06:02 a-25

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

tejasd avatar Sep 29 '22 17:09 tejasd

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.

a-25 avatar Nov 19 '22 17:11 a-25

Please close, merge, or comment. We plan to close stale PRs on November 28, 2023.

paulb777 avatar Nov 14 '23 00:11 paulb777

@ryanwilson, @tejasd: what do you think, should it be merged?

a-25 avatar Nov 14 '23 07:11 a-25

Looks like no immediate plans to move forward on this. I'll close and we can reopen if plans change.

paulb777 avatar Dec 09 '23 01:12 paulb777