-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Expense preview - Preview gets duplicated on submit 2 duplicate expenses if any already exist #52343
Comments
Triggered auto assignment to @AndrewGable ( |
💬 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:
|
Production: bandicam.2024-11-11.22-53-03-766.mp4 |
I think this is a backend bug, I can still repro if I check out NewDot's production version locally, and can no longer repro if I check out Web-E's production branch. |
This is caused by https://github.com/Expensify/Web-Expensify/pull/44299, cc @yuwenmemon. Removing the DeloyBockerCacsh label. |
Hmmmm... This only seems to happen when you are the workspace's admin in addition to the submitter.. |
Ah, we're not removing the optimistic report action, and updating the old one that actually exists... |
I have a fix but will create a revert the offending PR in the meantime so we can move forward with the deploy if we need to before the next Auth deploy cc @mountiny |
Asked for a retest here https://expensify.slack.com/archives/C9YU7BX5M/p1731434051476589 |
Not able to reproduce 2024-11-12.19-02-08.mp4 |
Is this the same bug: Two previews are created after creating 2 or more expenses v9.0.66-8bug_production.movcc: @yuwenmemon @mountiny Thank you for your help! |
@brunovjk yes that seems like a bug |
Can you report in Slack if its not reported yet? |
Yes, I saw it reported there just after this message from you |
Great then :D |
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: 9.0.60-0
Reproducible in staging?: Y
Reproducible in production?: N
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: N/A
Issue reported by: Applause - Internal Team
Issue found when executing PR #52060
Action Performed:
Expected Result:
One expense preview should be displayed
Actual Result:
Duplicate expense preview appears after submitting duplicate expenses
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6661817_1731357818722.2024-11-11_21-20-22.mp4
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: