-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[Due for payment 2025-04-28] [Due for payment 2025-04-25] Onboarding - Infinite Loading when logging in by adding a existing private domain #60397
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 @mollfpr ( |
Triggered auto assignment to @greg-schroeder ( |
💬 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:
|
@mollfpr are you able to reproduce this one?, i tested and can’t |
Checking 👀 |
@allgandalf I can reproduce the issue. You must use an email with a private domain that has already been validated previously. Screen.Recording.2025-04-17.at.22.35.08.mp4After I refresh the page, it will show the Concierge report from the merged account. |
So there is a workaround, refreshing the page, which fixes the issue at least. Weird bug though 🐛 |
I don't see any obvious errors coming from the backend that would cause this, so I suspect this will be a frontend fix, but still very strange. |
Checking now |
↪ Looking at this video seems like the |
okay, i get this one, the reportID of the concierge is still that of the old public email, on FE we need to update to save the report ID of the new concierge chat!, looking at that logic now! |
Alright, what happened here was that: Originally we do not pass any reportID to
Now the last accessed report in this case was the concierge report of the old (public email) account, but the new (work email) account no longer has access to the old accounts To fix it, we pass the concierge chat reportID to |
Nice job figuring that out! |
Fix is CP-ed to staging, asking for retest |
Reproducible in 9.1.29-9 Screen.Recording.2025-04-18.at.7.34.16.AM.mov |
@kavimuru i think this now has to do with the layout changing, the navigation bar changed it’s position from the bottom to left, so i guess this is only reproducible for accounts on that beta and real users won’t face this, as @ikevin127 and i both were able to see concierge chat with latests changes! |
Yeah, I wanted to mention the new app layout update and most likely it's a regression from there - not related to private domain onboarding. |
Screen.Recording.2025-04-18.at.10.03.06.mp4I confirm the issue is fixed. @kavimuru Could you retest with private domain without beta? |
QA team checked with a private domain without beta, but the flow is stuck. bandicam.2025-04-18.10-26-42-453.mp4 |
@IuliiaHerets, from the video i can see that you are facing this issue, it's a know BE bug, what you need to do is that once you enter a email, please complete that onboarding flow, then signup again with a different email, don't change the work email twice on the same screen. can you please retest this way and confirm if the bug has been fixed |
@allgandalf I checked with another temporary email and it works well bandicam.2025-04-18.10-59-02-621.mp4 |
nice, @mollfpr good to close then! |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.29-10 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2025-04-25. 🎊 |
@mollfpr @greg-schroeder @mollfpr The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the BugZero Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button] |
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.30-4 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2025-04-28. 🎊 |
@mollfpr @greg-schroeder @mollfpr The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the BugZero Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button] |
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: v9.1.29-0
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail: #54186
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team
Device used: Mac 14.5/ Chrome
App Component: Chat Report View
Action Performed:
Expected Result:
No infinite loading on the chat
Actual Result:
Infinite loading on the chat
Workaround:
Unknown
Platforms:
Screenshots/Videos
Bug6804452_1744850762163.Screen_Recording_2025-04-17_at_3.27.31_at_night.mp4
View all open jobs on GitHub
Issue Owner
Current Issue Owner: @greg-schroederThe text was updated successfully, but these errors were encountered: