-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[Due for payment 2025-05-20] [$500][HIGH] Approve button not showing up on reports even though need to approve them #60030
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 @RachCHopkins ( |
This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989 |
Report Previews are behind a Beta. |
Turns out this is not the new "report previews" feature, just the bog standard preview. (no beta required) Took me a while to repro, and not with my test reports - I had an old one that was in a workspace chat where the Approve button actually vanished while I was looking at it, and it didn't come back until I clicked in and back out of it. I had another which said Submit, and after I clicked it said Approve, then flicked back to Submit and then back to Approve in quick succession. Sadly it doesn't seem to do these things while I am recording the screen. |
Job added to Upwork: https://www.upwork.com/jobs/~021911598661603688104 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @rojiphil ( |
Triggered auto assignment to @youssef-lr ( |
Upwork job price has been updated to $500 |
Starting at HIGH and added AutoAssignerNewDotQuality. @youssef-lr can you review and provide feedback on what you think might be going on? Bumped to $500 in case a contributor might be able to figure it out. |
@mallenexpensify The child report related to expense is not loaded when we open the workspace chat and we need the report here to check if the user can approve it. Was there a recent backend change related to this? |
Thanks @jaydamani , @youssef-lr can you review plz? Thx |
@rojiphil Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
📣 @linhvovan29546 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
Issue not reproducible during KI retests. (Second week) |
Issue is not reproduced using these repro steps with Gmail accounts bandicam.2025-05-13.10-47-07-118.mp4 |
@IuliiaHerets Looks like the beta code has gone into production which makes the current steps invalid. We will have to modify the steps as follows:
Production - Issue Exists60030-production-issue.mp4Staging - Issue Fixed via PR60030-staging-fixed.mp4 |
@rojiphil Issue still not repro on staging https://github.com/user-attachments/assets/e0cdb992-c569-4b1a-a18e-adee670654d1 bandicam.2025-05-13.16-04-13-221.mp4 |
@IuliiaHerets That's great news. So, we are on the same page. |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.44-8 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-05-20. 🎊 For reference, here are some details about the assignees on this issue:
|
@rojiphil @RachCHopkins @rojiphil 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] |
Checklist please @rojiphil ! |
BugZero Checklist:
Bug classificationSource of bug:
Where bug was reported:
Who reported the bug:
Regression Test ProposalPrecondition:Test:
Do we agree 👍 or 👎 |
@RachCHopkins BZ Checklist done. Thanks. |
Payment Summary:
Upwork job here |
Contributor has been paid, the contract has been completed, and the Upwork post has been closed. |
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:
Reproducible in staging?: Needs Reproduction
Reproducible in production?: Needs Reproduction
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
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: @danielrvidal
Slack conversation (hyperlinked to channel name): #Quality
Action Performed:
Expected Result:
Approve button displayed in report preview
Actual Result
The Approve button is not displayed on the report preview initially. However, it appears in the report details view when clicked. Additionally, navigating to the expense chat causes the Approve button to show up on the report preview as well.
Workaround:
Unknown
Platforms:
Select the officially supported platforms where the issue was reproduced:
Platforms Tested:
On which of our officially supported platforms was this issue tested:Screenshots/Videos
Add any screenshot/video evidence
The video for the issue, logs and profile trace are in the OP
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @Issue Owner
Current Issue Owner: @RachCHopkinsThe text was updated successfully, but these errors were encountered: