App icon indicating copy to clipboard operation
App copied to clipboard

[Wave 8] [Ideal Nav] Incorrect or missing brick road indicators

Open hayata-suenaga opened this issue 1 year ago • 9 comments

Action Performed:

GBR (Green brick road) and RBR (Red brick road) are not working with the introduction of Ideal Navigation (the new navigation structure). This is a widespread error. Please check major GBR and RBR flows. They're most likely broken.

Some example of GBR and RBR not working:

  • Some GBR and RBR are incorrectly displayed on on Workspace Switcher when there is no report with GBR and RGR in the workspace. https://github.com/Expensify/App/pull/33280#issuecomment-1911253960
  • RBR is incorrectly displayed on Profile Avatar https://github.com/Expensify/App/pull/33280#issuecomment-1880199054

Expected Result:

GBR and RBR should work.

Actual Result:

GBR and RBR are not working correctly.

Workaround:

N/A

Platforms:

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

  • [x] Android: Native
  • [x] Android: mWeb Chrome
  • [x] iOS: Native
  • [x] iOS: mWeb Safari
  • [x] MacOS: Chrome / Safari
  • [x] MacOS: Desktop

Screenshots/Videos

The links to videos are in the Action Performed section above.

View all open jobs on GitHub

hayata-suenaga avatar Feb 01 '24 23:02 hayata-suenaga

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

melvin-bot[bot] avatar Feb 01 '24 23:02 melvin-bot[bot]

👋 Seems like there might be an issue with rows appearing unread as well: https://expensify.slack.com/archives/C036QM0SLJK/p1706888366992169

trjExpensify avatar Feb 02 '24 17:02 trjExpensify

@hayata-suenaga - is this one ready to assign external or should it remain internal?

alexpensify avatar Feb 02 '24 22:02 alexpensify

I think there is someone from SWM already working on this. Let me check.

hayata-suenaga avatar Feb 02 '24 23:02 hayata-suenaga

Does this issue still occur? I've added many fixes to displaying RBR/GBR since reporting this issue

WojtekBoman avatar Feb 05 '24 07:02 WojtekBoman

On the staging after the feature branch (the big PR) was merged, several issues were reported about RBR/GBR.

@WojtekBoman did you merge any PR after the feature branch was merged?

hayata-suenaga avatar Feb 05 '24 17:02 hayata-suenaga

I haven't merged any PRs since the feature branch was merged, but this issue https://github.com/Expensify/App/pull/33280#issuecomment-1880199054 should no longer appear :) I had fixed it, before the ideal nav was merged to the main. Additionally this PR contains the fix for displaying RBR/GBR.

WojtekBoman avatar Feb 06 '24 11:02 WojtekBoman

thank you for letting me know about the work you did 🙇 I apologize I wasn't aware of the previous fixes you worked on.

I had a chance to try reproducing the the issue. When there is an money request that needs fix, I didn't see the RBR on the workspace list item nor on the chat list item.

Screenshot 2024-02-06 at 9 44 51 AM 1 Screenshot 2024-02-06 at 9 52 06 AM

hayata-suenaga avatar Feb 06 '24 17:02 hayata-suenaga

Tested this just now and it seems working as expected.

s77rt avatar Feb 06 '24 22:02 s77rt

nice! I'll close this issue for now. Let's re-open this if the same issue appears again.

hayata-suenaga avatar Feb 08 '24 18:02 hayata-suenaga