-
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
[HOLD for payment 2024-03-05][$500] "Delete Request" and "Add Recipient" options persist after deletion until page refresh #36496
Comments
Job added to Upwork: https://www.upwork.com/jobs/~016b309ef80952eec2 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @jjcoffee ( |
Triggered auto assignment to @Christinadobrzyn ( |
We think that this bug might be related to #vip-vsp |
ProposalPlease re-state the problem that we are trying to solve in this issue.The "delete" and "add receipt" options do not disappear from the 3-dot menu. What is the root cause of that problem?This is happening because we are memoizing the ReportScreen without taking into account the When we remove the money request, the App/src/components/MoneyRequestHeader.tsx Line 72 in 2e3840f
So after the parentReportAction is deleted, we shouldn't be able to modify the money request, and therefore the buttons won't be present in the 3-dot menu. However, since the ReportScreen is memoized incorrectly, the What changes do you think we should make in order to solve the problem?We need to add a check to compare the _.isEqual(prevProps.parentReportAction, nextProps.parentReportAction) App/src/pages/home/ReportScreen.js Lines 661 to 675 in 2e3840f
Result:Screen.Recording.2024-02-14.at.16.01.36-compressed.mp4What alternative solutions did you explore? (Optional) |
@paultsimura's proposal LGTM! 🎀👀🎀 C+ reviewed |
Triggered auto assignment to @cristipaval, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
📣 @jjcoffee 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app! |
📣 @paultsimura 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
Thanks. The PR is ready for review: #36546 |
Deployed to production: #36546 (comment) |
Regression Test Proposal
Do we agree 👍 or 👎 |
Thanks for tagging me in this @paultsimura! Payouts due: Contributor: $500 @paultsimura (paid in Upwork) Upwork job is here. Regression test posted - https://github.com/Expensify/Expensify/issues/376258 Closing this! let me know if I'm missing anything! |
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: 1.4.41-2
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): abebemihiret@gmail.com
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Expected Result:
"Delete Request" and "Add Recipient" options should be immediately removed after deleting the request
Actual Result:
The options persist until the page is refreshed or navigated away from and then back to
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6379322_1707914136239.Screen_Recording_2024-02-14_at_3.46.47_AM.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: