-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[HOLD for payment 2025-04-14] [Navigation][HIGH] Chat - User lands on inbox before being redirected to new created chat or group. #56799
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
Comments
Triggered auto assignment to @techievivek ( |
Triggered auto assignment to @slafortune ( |
💬 A slack conversation has been started in #expensify-open-source |
👋 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:
|
Let me check this out |
I believe this is an expected behaviiour right now, it is different than before. We can discuss this with @WojtekBoman and @adamgrzybowski |
It would be good to directly navigate to the group chat. |
Here is the discussion we had https://swmansion.slack.com/archives/C01GTK53T8Q/p1738770962166749?thread_ts=1737562116.657899&cid=C01GTK53T8Q |
Hey, I'm going to work on that :) |
Making this weekly, its not that urgent and Wojtek will be ooo |
I'm working on it! |
Hey, to fix this issue I need changes from this PR, I'll raise a PR for this issue once the PR I mentioned is merged |
@WojtekBoman looks like you're good to go now 👍 |
This issue has not been updated in over 15 days. @slafortune, @mountiny, @WojtekBoman, @ZhenjaHorbach 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! |
@WojtekBoman This was resolved right and we can close? |
Yes, I see the PR for that has been merged |
@slafortune I think we just need to pay $250 to @ZhenjaHorbach now No new regression tests are required for this imho @ZhenjaHorbach can you complete the checklist please? |
BugZero Checklist:
Bug classificationSource of bug:
Where bug was reported:
Who reported the bug:
Unfortunately, I did not find a suitable PR.
Regression Test ProposalTest:
Do we agree 👍 or 👎 |
@ZhenjaHorbach Offer sent here - https://www.upwork.com/nx/wm/offer/106916806 |
@ZhenjaHorbach Paid ✅ |
Uh oh!
There was an error while loading. Please reload this page.
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number: 9.0.98-0
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: No, reproducible on hybrid only
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team
Device used: Motorola MotoG60 - Android 12 - Chrome
App Component: Chat Report View
Action Performed:
Expected Result:
User should be automatically redirected to new individual or group chat after finishing the creation.
Actual Result:
User lands on inbox before being redirected to the new created individual or group chat.
Workaround:
Unknown
Platforms:
Screenshots/Videos
Bug6741316_1739402227074.Create.mp4
View all open jobs on GitHub
Issue Owner
Current Issue Owner: @slafortuneThe text was updated successfully, but these errors were encountered: