Revert "[navigation-refactor] Fix browser history for complex navigation actions"

[mountiny]

Reverts #24165

[shubham1206agra]

This is happening due to multiple race conditions between window.history methods and the onStateChange method execution. Due to which, multiple window.history.go(-1) calls were made, and #25527 occurs

[shubham1206agra]

This is happening due to multiple race conditions between window.history methods and the onStateChange method execution. Due to which, multiple window.history.go(-1) calls were made, and #25527 occurs

I have fixed this issue. Can I put the patch PR?

[mountiny]

https://25572.pr-testing.expensify.com/

[OSBotify]

🧪🧪 Use the links below to test this build in android and iOS. Happy testing! 🧪🧪

android 🤖	iOS 🍎
https://ad-hoc-expensify-cash.s3.amazonaws.com/android/25572/index.html	https://ad-hoc-expensify-cash.s3.amazonaws.com/ios/25572/index.html
desktop 💻	web 🕸️
https://ad-hoc-expensify-cash.s3.amazonaws.com/desktop/25572/NewExpensify.dmg	https://25572.pr-testing.expensify.com

[hayata-suenaga]

@mountiny are you gonna revert the revert?

[mountiny]

@hayata-suenaga not sure what you mean, I have created this revert branch to make sure if it would fix the issues but I have confirmed the issues linked to the Pr still prevailed so it was not the root cause. No need to revert

[hayata-suenaga]

@mountiny I have an issue that was being held for the original PR. Since you reverted the original PR (but confirmed that it was not the root cause of an issue), I was wondering if you're planning to bring back the original PR by reverting this revert PR.

[mountiny]

Probably some miscommunication, the PR was not reverted. This Revert PR has not been merged so the original PR is still there

[hayata-suenaga]

ahh can't believe I missed this big red sign 😓 thank you for much for the clarification 🙇

[mountiny]

So not only Big Green buttons but also big red signs

[hayata-suenaga]

at this point, I'm terrified of any buttons and signs in general 😆