Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.1k stars 2.6k forks source link

Wrong bottom navigation icon is highlighted #44512

Open m-natarajan opened 4 days ago

m-natarajan commented 4 days ago

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Version Number: 9.0.2-2 Reproducible in staging?: y Reproducible in production?: y If this was caught during regression testing, add the test name, ID and link from TestRail: Email or phone of affected tester (no customers): Logs: https://stackoverflow.com/c/expensify/questions/4856 Expensify/Expensify Issue URL: Issue reported by: @allgandalf Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1719437029378469

Action Performed:

  1. Open app
  2. Go to search tab
  3. Click on any existing report
  4. Click on the subtitle
  5. Press back button
  6. Press back button again

Expected Result:

User should be on the search page

Actual Result:

the user is on the Chat page, but in bottom navigation, search tab is highlighted

Workaround:

unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

Screenshots/Videos

https://github.com/Expensify/App/assets/38435837/86741512-a789-456f-a2b0-eea68fc2f080

https://github.com/Expensify/App/assets/38435837/57b5776f-7369-481c-861f-c6796f4abdc7

View all open jobs on GitHub

melvin-bot[bot] commented 4 days ago

Triggered auto assignment to @puneetlath (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

allgandalf commented 4 days ago

[!NOTE] @puneetlath , can i be the C+ here, as i reported this issue and have some context of the same

mountiny commented 4 days ago

@WojtekBoman @adamgrzybowski this seems to be another task good for you

adamgrzybowski commented 4 days ago

@mountiny I have pending PR for this bug. But it seems to have some problems after merging the newest main. I will investigate it after other high priority issues