-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
[Navigation] Workspace-Overview page is shown briefly before landing on memebers page when inviting a user #56871
Comments
Triggered auto assignment to @johncschuster ( |
🚨 Edited by proposal-police: This proposal was edited at 2025-02-14 23:57:32 UTC. ProposalPlease re-state the problem that we are trying to solve in this issue.Workspace-Overview page is shown briefly before landing on memebers page when inviting a user What is the root cause of that problem?this happens because we dismiss the invitation modal before navigating to members page : App/src/pages/workspace/WorkspaceInviteMessagePage.tsx Lines 115 to 117 in fe5b1e3
What changes do you think we should make in order to solve the problem?if we're on small screen we can prevent dismissing the modal and navigate directly to Members Page :
we could update WORKSPACE_MEMBERS Route to accept a backTo param to make navigation consistent with browser/device back button. as if we don't dismiss the modal it will stay in the navigation state leading to inconsistent behaviors so we'll need to clear it out either by clearing the invite screens from navigation or backto param What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?UI issue What alternative solutions did you explore? (Optional) |
|
ProposalPlease re-state the problem that we are trying to solve in this issue.Overview page is shown briefly before landing on memebers page when inviting a user What is the root cause of that problem?We dismiss the modal here, which pops previous screen before navigating to members page
What changes do you think we should make in order to solve the problem?As we are navigating to a central screen through a direct route, we can remove the following line
What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?None What alternative solutions did you explore? (Optional)None |
Hey, I'm going to work on that :) |
@johncschuster, @mountiny, @WojtekBoman Whoops! This issue is 2 days overdue. Let's get this updated quick! |
Making this weekly, its not that urgent and Wojtek will be ooo |
@johncschuster @mountiny @WojtekBoman 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! |
@WojtekBoman will be back next week |
Bumping for Melvin. We're waiting on @WojtekBoman to return from 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 |
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: v9.0.98-5
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Yes, reproducible on both
If this was caught during regression testing, add the test name, ID and link from TestRail: #56827
Email or phone of affected tester (no customers): htad26+dfsdjhfjhsdf@gmail.com
Issue reported by: Applause Internal Team
Device used: iPhone 13/iOS 18.3, Tecno Spark 8/Android 11
App Component: Workspace Settings
Action Performed:
Expected Result:
User lands on members page after finishing the invite a member flow and the overview page is not shown
Actual Result:
Overview page is shown briefly before the user lands on the members page
Workaround:
Unknown
Platforms:
Screenshots/Videos
Bug6742861_1739535269092.IMG_0834.mp4
View all open jobs on GitHub
Issue Owner
Current Issue Owner: @WojtekBomanThe text was updated successfully, but these errors were encountered: