-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Sign in – Unable to login via Magic link, Magic code expired appears #48378
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 @muttmuure ( |
Triggered auto assignment to @danieldoglas ( |
👋 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:
|
Can't reproduce, tried multiple times Screen.Recording.2024-09-02.at.2.10.02.AM.mov |
@IuliiaHerets can you try this one again? Looks like you were logged in on a tab already on your video, so maybe that's why? |
not a deploy blocker for the API since it doesn't happen in production |
@danieldoglas Issue is still reproducible, build - v9.0.27-0 Screenrecorder-2024-09-02-20-39-56-247.mp4 |
We are awaiting proposals |
@ishpaul777 are you able to reproduce this now? |
Lol no, my instinct kicked in and i wrote "awaiting proposal", what i mean is waiting reliable steps to repro Screen.Recording.2024-09-05.at.1.38.57.AM.mov |
I just ran into this on staging by sending an Invoice from one user to an email address that does not yet have an Expensify account. I was able to reproduce a second time by sending to another email without an account. |
nice, thanks for that @deetergp . @ishpaul777 can you try those steps and confirm it still fails? Then we can open it for external proposals. |
Seems we have a existing issue that we can not send a invoice to any user. https://expensify.slack.com/archives/C01GTK53T8Q/p1725613866403789 |
@danieldoglas, according to this thread I'll be taking this over as C+ from @ishpaul777 Could you re-assign this please?
I haven't looked into this too closely, could be either of those just by looking at the code, but I suspect this is a back-end issue. Let's not make it External just yet, I'll look into this closer and post my findings tomorrow |
Done, assigning @eVoloshchak to it! |
Hi @CortneyOfstad No adjustment to the reproducible steps. It is stell reproducible according steps. Build v9.1.14-4 Recording.3017.mp4 |
It's not clear whether it's purely a back-end issue, should we open this for proposals? |
@eVoloshchak you said you were going to post your findings, what were those? |
Wasn't able to find anything, we display the |
Hey @danieldoglas @eVoloshchak! I am going to be heading OoO this afternoon so if this needs BZ action, please re-assign the label. Otherwise I will review when I get back! |
Back from OoO — @eVoloshchak and @danieldoglas — where are we at with this? |
bump @eVoloshchak . Anything else you could find? |
@danieldoglas, traced this back to ![]() We do send the correct account ID and validation code, should be working ![]() This is probably a BE issue, I suggest we make this Internal |
Interesting - thanks for sending the requestID, that was very helpful. I can see that we received the validate code correctly, but looks like it didn't match the one we had available in the DB. I'll investigate more. |
Thanks @danieldoglas! |
Hey @danieldoglas — just checking in to see if there were any updates on this. Thanks! |
I think I know what's kind of going on here, but I'll need some time to discuss. Changing this to weekly while I finish some other pressing matters, but it's on my radar. |
Sounds good — thanks @danieldoglas! |
Hey @danieldoglas — just checking to see if we have a rough ETA when this will be looked at? Thanks! |
Bump @danieldoglas ^^^ thanks! |
Still on my list, but not a huge priority compared to some other tasks. I'm going to try to prioritize this in the next week |
Sounds good @danieldoglas — thank you! |
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: 9.0.27-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: https://expensify.testrail.io/index.php?/tests/view/4906568
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team
Action Performed:
Expected Result:
Abracadabra page opens. We have KI #44600
Actual Result:
Magic code expired appears
Workaround:
Unknown
Platforms:
Screenshots/Videos
Bug6588552_1725081784278.magic_link.mp4
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: