Daniel Gale-Rosen
Daniel Gale-Rosen
I think that's okay? Since potentially they could try the other option. I'm not sure, @Expensify/design what do you think we should do here? Should we hide the pop up...
okay cool, I'll dismiss the pop up when we close the modal. Thanks!
popup will now be dismissed - ready for another review @abdulrahuman5196
fixed the conflicts but the logic broke a bit, I need to figure out why - on (attempting to) close of the NoAccess modal, first the delegate menu closes, and...
Oh I think maybe it's the focus trap that's part of the added `PopoverMenu`. Do you know if there's a way to override that? It doesn't look like there's any...
ah okay let me try that. Also a million more conflicts, too. Sigh.
Okay welp in investigating the conflicts - I found [this](https://github.com/Expensify/App/pull/52103), which was just merged. We might not need this PR at all. Sigh. I'm going to test fully but seems...
Okay yeah it's a little different but seems to work equivalently. I'm going to close this out - thank you for your expertise and sorry for your trouble!
Okay, this looks like it works properly with https://github.com/Expensify/Mobile-Expensify/pull/13289 and the web-e / backend changes!
thanks @mollfpr! Yes unfortunately not something external folks can help with yet. @blimpich - let me know if you have any questions!