Repository navigation
Web - Inbox - User has to click "Continue setup" CTA twice in olddot to display "Connect bank account" page in the right sidebar #8295
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!
Issue was found when executing #8257
Action Performed:
- Go to OldDot and sign up for a new account with an @gmail.com address
- Click on the "Get started" on the inbox task that says "Would you like to get started with our free plan?".
- Sign out of NewDot
- Go back to the OldDot inbox and refresh the page
- Click "Continue setup" on the inbox task that says "Finish setting up your bank account".
Expected Result:
The Connect bank account page should open in the right sidebar
Actual Result:
The Connect bank account page opens only on second click and refresh page.
Workaround:
Unknown
Platform:
Where is this issue occurring?
- Web
Version Number: v1.1.44-5
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
Bug5504272_continue2.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause
Slack conversation:
View all open jobs on GitHub
Issue was found when executing #8257
Activity
melvin-bot commentedon Mar 24, 2022
Triggered auto assignment to @MariaHCD (
Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.[-]Web - Inbox - User has to click "Continue setup" CTA in olddot to display "Connect bank account" page in the right sidebar[/-][+]Web - Inbox - User has to click "Continue setup" CTA twice in olddot to display "Connect bank account" page in the right sidebar[/+]MariaHCD commentedon Mar 28, 2022
From the logs, the call to GetShortLivedAuthToken here succeeded.
However, looks like the first few attempts to navigate to the bank account page didn't work (logs):
Which is why we redirected back to OldDot. But it looks like the test account is on the free plan beta. Not sure what went wrong here.
cc: @TomatoToaster @madmax330 any ideas?
MariaHCD commentedon Mar 31, 2022
Unable to reproduce the issue locally but I am able to reproduce on production with a gmail account (logs). With the following steps:
It should be noted that if signed into both OldDot and NewDot, the transition works as expected.
melvin-bot commentedon Mar 31, 2022
Current assignee @MariaHCD is eligible for the Demolition assigner, not assigning anyone new.
15 remaining items
melvin-bot commentedon Apr 18, 2022
@neil-marcellini Whoops! This issue is 2 days overdue. Let's get this updated quick!
neil-marcellini commentedon Apr 18, 2022
I'm updating the PR for this issue to merge into the feature branch neil-transition-flows. Once all of the transition flows are working I will put up a PR to merge the feature branch into main.
melvin-bot commentedon Apr 26, 2022
@neil-marcellini Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!
melvin-bot commentedon Apr 28, 2022
@neil-marcellini Huh... This is 4 days overdue. Who can take care of this?
neil-marcellini commentedon Apr 28, 2022
Under review and working on the transitions.
melvin-bot commentedon May 6, 2022
@neil-marcellini Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!
neil-marcellini commentedon May 6, 2022
Still under review for this Onyx PR Set default key states when clearing the cache.
melvin-bot commentedon May 16, 2022
@neil-marcellini Eep! 4 days overdue now. Issues have feelings too...
neil-marcellini commentedon May 16, 2022
I'm still working on this, trying to get this new Onyx PR merged in.
mvtglobally commentedon May 17, 2022
Issue not reproducible during KI retests. (First week)
mvtglobally commentedon May 24, 2022
Issue not reproducible during KI retests. (Second week)
melvin-bot commentedon May 24, 2022
@neil-marcellini Whoops! This issue is 2 days overdue. Let's get this updated quick!
neil-marcellini commentedon May 24, 2022
Sounds like this is currently not reproducible and I believe the previous PRs linked here have fixed it. I'm going to close this.