Expensify
[BE - Requires Internal Engineer] [$250] Web - Hold - Red dot is lost on held expense when approve pending amount
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: v9.0.48-2 Reproducible in staging?: Y Reproducible in production?: Y Issue was found when executing this PR: https://github.com/Expensify/App/pull/49910 Email or phone of affected tester (no customers): [email protected] Issue reported by: Applause Internal Team
Action Performed:
- Log in with your Approver account
- As employee Go to the room with the two expenses
- As employee: hold one expense
- Verify there is a red dot on held expense on both sides
- As Approver: Approve just the pending amount
- As Approver: back to room by click to the header
Expected Result:
Red dot should remain on held report when Approver go back to room
Actual Result:
Red dot disappears from held expense when approve pending amount. It appears only when open the held report again and navigate back to room
Workaround:
Unknown
Platforms:
- [ ] Android: Native
- [ ] Android: mWeb Chrome
- [ ] iOS: Native
- [ ] iOS: mWeb Safari
- [x] MacOS: Chrome / Safari
- [ ] MacOS: Desktop
Screenshots/Videos
https://github.com/user-attachments/assets/42af07e2-719e-4679-a7e8-5a7c1dbea52b
Upwork Automation - Do Not Edit
- Upwork Job URL: https://www.upwork.com/jobs/~021846971961812039151
- Upwork Job ID: 1846971961812039151
- Last Price Increase: 2024-11-28
Issue Owner
Current Issue Owner: @ntdiary
Triggered auto assignment to @zanyrenney (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.
![melvin-bot[bot] avatar](https://avatars.githubusercontent.com/in/179547?v=4 "Published by a pub.dev verified publisher"){kind=small}
@zanyrenney FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors
Job added to Upwork: https://www.upwork.com/jobs/~021846971961812039151
Triggered auto assignment to Contributor-plus team member for initial proposal review - @ntdiary (External)
Quick question, how do we make a account approver and another employee? Thanks for any help.
@ntdiary, @zanyrenney Eep! 4 days overdue now. Issues have feelings too...
https://github.com/Expensify/App/blob/4f0ca943a51e340652885c3c1fa3dd4434b91e3f/src/components/ReportActionItem/ReportPreview.tsx#L156-L162
When Approver goes back to room, the hold violation data doesn't exist yet, so ReportPreview will not display RBR, that data will return by server if approver opening the held report.
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸
@ntdiary, @zanyrenney Eep! 4 days overdue now. Issues have feelings too...
This issue requires an internal engineer. When an employee puts an expense on hold, our app doesn't push the hold violation data to the approver, so their ReportPreview component won't display RBR.
Unless ReportPreview also checks the hold field in the transaction_xxx like the MoneyRequestPreview does to display RBR.
cc @zanyrenney
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸
@ntdiary, @zanyrenney Huh... This is 4 days overdue. Who can take care of this?
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸
@ntdiary, @zanyrenney 12 days overdue now... This issue's end is nigh!
We require an internal engineer here. I have applied the lable
Eh, @zanyrenney, it seems that an internal engineer hasn't been assigned automatically? 😂
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸
@ntdiary, @zanyrenney Eep! 4 days overdue now. Issues have feelings too...
it seems that an internal engineer hasn't been assigned automatically? 😂 That's working as expected.
Seeing as we have said expense is lower priority, I am going to move this to weekly. It is tagged correctly in expense.
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸
I actually think this fits in a cohort room since it definitely affects our current cohort. I've moved to #convert since we're pitching Hold to new users.
This issue has not been updated in over 15 days. @ntdiary, @zanyrenney eroding to Monthly issue.
P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!
@ntdiary, @zanyrenney, this Monthly task hasn't been acted upon in 6 weeks; closing.
If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead.