Repository navigation
[Ideal Nav] WS switcher - Delayed transition of breadcrumbs from Workspace / Chat to Expensify / Chat #38281
Description
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: 1.4.52-0
Reproducible in staging?: Y
Reproducible in production?: N
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:
Precondition:
- There are at least two workspaces.
- One of the workspaces has other members,
- Go to staging.new.expensify.com.
- Open workspace switcher > Select a workspace.
- Go to Profile > Workspaces.
- Select a different workspace that has members.
- Go to Members.
- Click on the invited members > Message .
Expected Result:
The breadcrumbs (Expensify / Chat) will sync with the changes in LHN.
Actual Result:
LHN shows all the chats first, then followed by the change of breadcrumbs (Workspace / Chat) to Expensify / Chat. The transition is not smooth.
Workaround:
Unknown
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
Activity
melvin-bot commentedon Mar 14, 2024
Triggered auto assignment to @alexpensify (
Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.github-actions commentedon Mar 14, 2024
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
melvin-bot commentedon Mar 14, 2024
Triggered auto assignment to @techievivek (
Engineering), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.izarutskaya commentedon Mar 14, 2024
@techievivek I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.
izarutskaya commentedon Mar 14, 2024
We think that this bug might be related to #wave-collect - Release 1
CC @trjExpensify
techievivek commentedon Mar 14, 2024
Doesn't need to be a blocker but I am checking the diff to figure out which PR could have caused this regression.
techievivek commentedon Mar 14, 2024
Also, I can see there is a slight delay but not sure if this is due to some recent PR.
72 remaining items
alexpensify commentedon May 17, 2024
Heads up, I will be offline until Tuesday, May 28, 2024, and will not actively watch over this GitHub during that period.
@trjExpensify - While I'm offline, I need help here completing the payment process after this one clears the 7-day period. That said, it looks like the PR created regression, so we need to address that asap. Thanks!
parasharrajat commentedon May 18, 2024
PR is merged.
melvin-bot commentedon May 24, 2024
@alexpensify, @trjExpensify, @parasharrajat, @techievivek, @WojtekBoman Whoops! This issue is 2 days overdue. Let's get this updated quick!
melvin-bot commentedon May 28, 2024
@alexpensify, @trjExpensify, @parasharrajat, @techievivek, @WojtekBoman 6 days overdue. This is scarier than being forced to listen to Vogon poetry!
trjExpensify commentedon May 28, 2024
Dropping off and handing this back to Al.
alexpensify commentedon May 28, 2024
Noted, I'm going through my BZ batch soon and was planning to grab this one back.
alexpensify commentedon May 28, 2024
Ok, I need to start the payment process tomorrow. Looking at the PR, it was merged on May 22.
alexpensify commentedon May 29, 2024
Alright, keeping the summary as is since this was a more complicated PR and was quickly addressed:
#38281 (comment)
@parasharrajat please submit the request in Chat.
parasharrajat commentedon Jun 14, 2024
Payment requested as per #38281 (comment)
JmillsExpensify commentedon Jun 17, 2024
$250 approved for @parasharrajat per this summary.