[Wave Collect] [Ideal Nav] Navigation among different Workspace Settings pages is broken

[hayata-suenaga]

Action Performed:

1. Test this on a browser
2. Go to Workspace Settings page (make sure that you’re on a workspace in which you’re an admin so that you have enough settings items to navigate through).
3. Click through different setting items (watch the video)
4. Shrink the screen
5. Press the browser back button several times.

Expected Result:

The back button should take you straight back to the Settings page where the settings items are listed.

Actual Result:

Clicking through the back button takes you to different settings item page previously visited.

Workaround:

Can the user still use Expensify without this being fixed? Have you informed them of the workaround?

Platforms:

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

• Android: Native
• Android: mWeb Chrome
• iOS: Native
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

#33280 (comment)

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~0112c8e18133890b6e
• Upwork Job ID: 1763217867286740992
• Last Price Increase: 2024-02-29

[melvin-bot]

Triggered auto assignment to @sakluger (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

[hayata-suenaga]

Might be fixed in this GH issue. I'm confirming this.

[sakluger]

@hayata-suenaga I didn't see any response on the other issue. Did you find out if they are related? And if they are, should we add a HOLD on this issue, or just close it out?

[hayata-suenaga]

I didn't see any response on the other issue. Did you find out if they are related?

asking directly on Slack

And if they are, should we add a HOLD on this issue, or just close it out?

is the issues turn out to be related, we can close this issue in favor of this one.

Let me get back to you once I get a response 👍

[adamgrzybowski]

Hey @hayata-suenaga, this one won't be solved by the goback workaround and may be tricky. I'll take a look at it when I have some free time. But maybe in the meantime, we could change it to weekly?

[melvin-bot]

@sakluger @hayata-suenaga this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

[hayata-suenaga]

Adam is going to handle this issue when they have time

[melvin-bot]

@sakluger @hayata-suenaga this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks!

[hayata-suenaga]

Adam is OOO and they will work on this once they're back. This issue will also take time to fix.

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~0112c8e18133890b6e

[melvin-bot]

Triggered auto assignment to Contributor Plus for review of internal employee PR - @c3024 (Internal)

[hayata-suenaga]

@adamgrzybowski is going to work on this. because the solution is a little bit tricky, it might take some time. Melvin, please chill 🙇

[sakluger]

Switching back to weekly. Let's see if Melvin lets it stay that way. 😂

[hayata-suenaga]

@adamgrzybowski is going to on this when they have time

[hayata-suenaga]

@WojtekBoman Let me know if #37359 is related

[hayata-suenaga]

@adamgrzybowski or @WojtekBoman is going to work on this.

[WojtekBoman]

I'm investigating that :)

[trjExpensify]

What are the next steps here? Is this on hold for the issue to refactor goBack logic?

[hayata-suenaga]

Thank you Tom for checking on the status

I re-tested this, and I was not able to repro. @WojtekBoman, could you double confirm this? If you can't also repro, we can close this issue.

[trjExpensify]

Cool, @WojtekBoman want to confirm and we can close?

[WojtekBoman]

Unfortunately, it seems to be still present:

Screen.Recording.2024-04-15.at.16.15.38.mov

We've spent some time on investigation this issue with @adamgrzybowski and it's a bit complex to fix, because we can't directly modify browser history. Fixing this issue might require many changes in the logic that works fine, and this case shouldn't be common for users. We had to switch to work on the new search tab and I'm wondering what should we do with this task, because as I mentioned it's not a common issue and handling that might require some interference with browser history and I'm not sure if we want to make such deep changes to deal with that.

[trjExpensify]

Is this specifically a bug on Web when you're shrinking your screen link that or was that just to replicate a native mobile experience?

[hayata-suenaga]

@trjExpensify I believe this is specific to the web.

I believe that we don't have to spend time on fixing this. The fix might also further complicates the navigation logic that is already hard to understand. I'd say let's close this issue, but I'll wait for @trjExpensify's response.

[trjExpensify]

Okay, then yeah.. let's close this if it's not simulating mobile!