-
Notifications
You must be signed in to change notification settings - Fork 3k
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 2023-12-01] [$500] Web – Login – New dot opens when navigate to the staging magic link, not abracadabra page. #30988
Comments
Job added to Upwork: https://www.upwork.com/jobs/~01941d72fc174377c3 |
Triggered auto assignment to @CortneyOfstad ( |
Bug0 Triage Checklist (Main S/O)
|
Triggered auto assignment to Contributor-plus team member for initial proposal review - @robertKozik ( |
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open
|
Triggered auto assignment to @nkuoch ( |
ProposalPlease re-state the problem that we are trying to solve in this issue.Web – Login – New dot opens when navigate to the staging magic link, not abracadabra page What is the root cause of that problem?In
we change the order of routes. In this case, the original order of state.routes is [Home, CenterPanelNavigator, ValidateLogin], but the routes after this logic is [Home, ValidateLogin, CenterPanelNavigator] -> The report screen is shown (last item is routes is CenterPanelNavigator) What changes do you think we should make in order to solve the problem?We should preserve the order of routes. In order to do that we can change the logic of splitRoutes to
and change stateToRender to
ResultScreen.Recording.2023-11-07.at.21.18.58.mov |
📣 @tienifr 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
📣 @situchan 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
The PR is up |
This still needs to be CPed @nkuoch |
This issue has not been updated in over 15 days. @nkuoch, @CortneyOfstad, @situchan, @tienifr 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! |
The automation failed here, @CortneyOfstad can you please pay $500 to @tienifr and @situchan |
@situchan it is showing in Upwork that it is waiting for you to accept. Please let me know once you complete that — thanks! |
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: 1.3.96-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause-Internal Team
Slack conversation: @
Action Performed:
Expected Result:
Verify the abracadabra page is displayed
Actual Result:
New dot opens when navigate to the staging magic link, not abracadabra page.
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6267091_1699351294403.not_abracadabra_page.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @CortneyOfstadThe text was updated successfully, but these errors were encountered: