-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Unable to open a single expense report as approver #44411
Comments
This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989 |
Triggered auto assignment to @muttmuure ( |
Triggered auto assignment to @NikkiWines ( |
Hmm, I'm not able to reproduce this issue on staging currently. @puneetlath can you confirm if this is still an issue for you? Screen.Recording.2024-06-26.at.23.28.47.mov |
Reached out to @puneetlath about this today - there's an ongoing investigation into the bug here. Will follow up with this starting next week 🙇 |
Investigation is ongoing |
@NikkiWines @muttmuure this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks! |
Investigation is still ongoing |
@NikkiWines @muttmuure if this is a CRITICAL item it should be daily, right? I'll update, but lmk if I am missing something! |
I was expecting this to be fixed by better gap detection which merged this week - is this definitely still happening? |
@puneetlath do you mind testing this again? |
This hit staging 3 days ago, and production yesterday: #41962 |
Still having the problem. We've narrowed it down to a problem when a personal card transaction gets merged with a receipt. @NikkiWines is working on the fix. |
Getting tests working for this now, hoping to have a PR up by EOD |
Hmm, actually what I had wasn't working, pivoting to another solution where we update the IOU reportAction after merging |
@puneetlath, @NikkiWines, @muttmuure Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
This is done and deployed. Just need someone to confirm it's working properly when using this out in the wild. |
@puneetlath, @NikkiWines, @muttmuure Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
This works. Going to go ahead and close it out. |
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:
Reproducible in staging?: Needs reproduction
Reproducible in production?: Needs reproduction
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: @puneetlath
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1718982172128879
Action Performed:
Expected Result:
Report opened without any issue
Actual Result:
Unable to open the report and Loading skeleton displayed
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: