Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.53k stars 2.88k forks source link

[HOLD for payment 2024-09-27] [$250] Onboarding modal is flashing briefly on transitioning to NewDot via ShortLivedAuthToken #49245

Closed trjExpensify closed 1 month ago

trjExpensify commented 1 month ago

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.35-0 Reproducible in staging?: Y Reproducible in production?: Y If this was caught during regression testing, add the test name, ID and link from TestRail: Email or phone of affected tester (no customers): (examples in the linked slack thread in #wave-collect. Logs: https://stackoverflow.com/c/expensify/questions/4856 Expensify/Expensify Issue URL: Issue reported by: @anmurali Slack conversation: https://expensify.slack.com/archives/C036QM0SLJK/p1726074674036609

Action Performed:

_Note: we're still trying to nail down the prerequisites when it comes to the onboardingNVP state, but our best guess is that it needs to be empty._

  1. Go to OldDot
  2. Click Support
  3. Click Concierge to be redirected to NewDot
  4. Observe the brief flash of the onboarding modal

Expected Result:

No onboarding modal is shown, user lands in the Concierge DM chat

Actual Result:

Onboarding route briefly flashes in the URL bar causing the onboarding modal to appear and disappear

Workaround:

N/A, the modal does disappear again.

Platforms:

Which of our officially supported platforms is this issue occurring on?

Screenshots/Videos

Customer vids from FullStory, so not uploading them here. Refer to these two slack chats in threads for the visual examples:

https://expensify.slack.com/archives/C036QM0SLJK/p1726079102789129?thread_ts=1726074674.036609&cid=C036QM0SLJK https://expensify.slack.com/archives/C036QM0SLJK/p1726079241599169?thread_ts=1726074674.036609&cid=C036QM0SLJK

View all open jobs on GitHub

Issue OwnerCurrent Issue Owner: @
Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021835709139570374195
  • Upwork Job ID: 1835709139570374195
  • Last Price Increase: 2024-09-16
melvin-bot[bot] commented 1 month ago

Triggered auto assignment to @twisterdotcom (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

trjExpensify commented 1 month ago

Apols for the noise, @twisterdotcom. GH wouldn't let me assign myself on creation. Swapped you out. 👍

blazejkustra commented 1 month ago

Hey! I’m Błażej Kustra from Software Mansion, and I’d like to work on this issue 😄

mountiny commented 1 month ago

PR merged

trjExpensify commented 1 month ago

Dope, thanks!

melvin-bot[bot] commented 1 month ago

Job added to Upwork: https://www.upwork.com/jobs/~021835709139570374195

melvin-bot[bot] commented 1 month ago

Current assignee @allgandalf is eligible for the External assigner, not assigning anyone new.

trjExpensify commented 1 month ago

PR is on staging. We can close it out when it hits prod, as no payments will be due here.

mountiny commented 1 month ago

@trjExpensify I believe we have to pay @allgandalf $250 for review and testing

trjExpensify commented 1 month ago

Ah yes, sorry. I missed that assignment. 👍

allgandalf commented 1 month ago

PR deployed to production yesterday

CyberAndrii commented 1 month ago

Can we revert it 🤔? https://github.com/Expensify/App/pull/49263#issuecomment-2359236679

mountiny commented 1 month ago

@allgandalf $250 for their review i believe

allgandalf commented 1 month ago

Chill melvin, only payment remains

trjExpensify commented 1 month ago

Payment summary as follows:

Offer sent

trjExpensify commented 1 month ago

Paid, closing!