-
Notifications
You must be signed in to change notification settings - Fork 2.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[HOLD for payment 2024-07-23] [$250] [Search v1] - App returns to LHN from transaction thread in Search while Search is highlighted #43407
Comments
Triggered auto assignment to @marcochavezf ( |
👋 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:
|
@marcochavezf FYI 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 |
We think that this bug might be related to #wave-collect - Release 1 |
Demoting to NAB. @adamgrzybowski @Kicu could you please take a look? |
Triggered auto assignment to @laurenreidexpensify ( |
Job added to Upwork: https://www.upwork.com/jobs/~01b7ac090e7d716f56 |
Okay, there was slight delay but it's ready 😄 @WojtekBoman is currently reviewing the changes |
Triggered auto assignment to @lschurr ( |
@lschurr handing this one over for parental leave cover. We're in review for the PR, so this will just be handling payment. thanks! |
This issue has not been updated in over 15 days. @luacmartins, @lschurr, @adamgrzybowski, @getusha eroding to Monthly issue. P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do! |
I think this one is done |
Yes, PR deployed to prod on Jul 16. Updating the issue. |
Payment summary:
|
$250 approved for @getusha |
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.81-3
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:
Expected Result:
If app is meant to return to the Inbox tab, the Inbox tab should be highlighted
Actual Result:
App returns to LHN (Inbox), but the Search tab is highlighted instead.
When selecting a workspace from the workspace switcher, the interface changes to Search
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6508403_1718040129733.RPReplay_Final1718039873.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @lschurrThe text was updated successfully, but these errors were encountered: