Eugene Voloshchak

Results 102 comments of Eugene Voloshchak

> @eVoloshchak I'm assuming that my proposal wasn't accepted, am I correct? No, your proposal is absolutely valid, as I said, the decision should come down to the expected behavior,...

> What does "wait" mean -- wait for what exactly? I think as long as the checkbox gets toggled (which it does?), we're good to go as is...? @flodnv, there...

I feel like this can cause a lot of regressions, this is technically a workaround, agree with @flodnv I think we should try to resolve the root cause of the...

Bug: reports are grayed out when offline even after connecting back to the internet once https://user-images.githubusercontent.com/9059945/202502532-7dadb956-640c-48c7-a405-f479bd6706c2.mov

## Reviewer Checklist - [x] I have verified the author checklist is complete (all boxes are checked off). - [x] I verified the correct issue is linked in the `###...

@tungmt You are right, this inconsistency is caused by [this](https://github.com/Expensify/App/blob/e434efa1c455620c510441628b94081e0f8a564e/src/libs/actions/Policy.js#L832) `Navigation.dismissModal()` However we can't remove it, it was [added](https://github.com/Expensify/App/pull/10949/commits/08e2218f4233c3db789da0f1a11d3a347e94ad0c) to fix an issue with transitions to existing workspaces from OldDot...

@Puneet-here, cool, I like your solution! It indeed fixes this issue And it also works correctly when transitioning from OldDot to NewDot, however, I verified this by calling `setUpPoliciesAndNavigate` manually...

Wasn't able to find this task in OldDot, asking in [slack](https://expensify.slack.com/archives/C01GTK53T8Q/p1668622366210179)

Got a response and was able to test, @Puneet-here's [proposal](https://github.com/Expensify/App/issues/12246#issuecomment-1312188539) looks good! 🎀👀🎀 C+ reviewed! cc: @kevinksullivan