Repository navigation
[$250] Add bulkHold option in App #62503
Description
Problem
With the new report view, we show a table of transactions and allow bulk actions. Right now, we're doing a loop to hold transactions, which generates N API calls and breaks 1:1:1.
Solution
Use the BulkHold API command
cc @Tony-MK
Upwork Automation - Do Not Edit
- Upwork Job URL: https://www.upwork.com/jobs/~021925275432454831980
- Upwork Job ID: 1925275432454831980
- Last Price Increase: 2025-05-21
- Automatic offers:
- Tony-MK | Contributor | 107716791
Activity
melvin-bot commentedon May 21, 2025
Triggered auto assignment to @OfstadC (
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.[-]Add bulkHold option in App[/-][+][$250] Add bulkHold option in App[/+]melvin-bot commentedon May 21, 2025
Job added to Upwork: https://www.upwork.com/jobs/~021925275432454831980
melvin-bot commentedon May 21, 2025
Triggered auto assignment to Contributor-plus team member for initial proposal review - @Ollyws (
External)luacmartins commentedon May 21, 2025
PR is in review
50 remaining items
OfstadC commentedon Jul 8, 2025
@Ollyws can you review? 😃
Tony-MK commentedon Jul 8, 2025
The PR was recently approved.
melvin-bot commentedon Jul 31, 2025
This issue has not been updated in over 15 days. @Ollyws, @OfstadC, @luacmartins, @Tony-MK 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!
OfstadC commentedon Aug 18, 2025
I think this can be closed out now - Please reopen if I missed something here!
Ollyws commentedon Aug 20, 2025
@OfstadC we are still working on a followup PR to this here.
luacmartins commentedon Sep 15, 2025
We're still working on the App PR. Although work is paused for now until we figure out the correct approach with the upcoming snapshot project
[-][Due for payment 2025-06-16] [Due for payment 2025-06-13] [$250] Add bulkHold option in App[/-][+][$250] Add bulkHold option in App[/+]luacmartins commentedon Jan 5, 2026
I think this is done.
Ollyws commentedon Jan 6, 2026
@luacmartins Were me and @Tony-MK due payment for the followup PR?
luacmartins commentedon Jan 6, 2026
I'm not sure. We moved that PR over to #70721 and I assume payment was processed via that issue/PR.