Org Web Adapter

Editing pages/mmpy_272_add_removed_by_to_bulk_hold_creation.org

:PROPERTIES:
:ID:       14612d27-6931-4bc7-9960-dfbfa2ad0f47
:END:
#+title: MMPY-272 Add removed_by to bulk hold creation

- source :: https://multimediallc.atlassian.net/browse/MMPY-272
- epic :: [[id:88adb9ec-ab52-4d87-8625-6cf72d01612c][MMPY-198 Compliance Holds, Fraud Holds, and Banned Payees]]
- tags ::  [[id:4b532967-a1d9-45f4-b1f0-9a02ad540ff0][Falcon Nest Project Story]] [[id:7c3bbb4e-d079-42c4-992c-3e76210911f4][Falcon Nest Project]]
- status ::  [[id:4d41d978-e56f-45c1-88d1-e2c76510972e][Status - Complete]]
- pull :: https://github.com/multimediallc/mm-pay-internal/pull/337

* Description

Ensure all locations where a hold can be removed is including a removed_by
field. This was noticed on bulk_update calls to holds, but an exhaustive search
wasn’t completed.

** DONE Find all places where holds are created :work:feature:holds:mmpy272:multimediallc:mmpy198:spike:
:PROPERTIES:
:ID:       cd87c490-6e09-72c8-6906-6aab14627cf9
:END:
** DONE Figure out how to get a user in the ApplicationHoldsManager :work:feature:holds:mmpy272:multimediallc:mmpy198:spike:
:PROPERTIES:
:ID:       6dd29e9e-5e30-6256-8d16-96f550f57b0b
:END:
** DONE Add tests for adding an admin source user to holds :work:feature:holds:mmpy272:multimediallc:mmpy198:tests:
:PROPERTIES:
:ID:       4b1c9c88-0fd1-ea59-69b1-3a7e006a209c
:END:
** DONE Clean up any PR issues and merge :work:feature:holds:mmpy272:mmpy198:multimediallc:
:PROPERTIES:
:ID:       e862f3a4-9330-2045-136f-d44f00146371
:END:
- Note taken on [2025-08-15 Fri 11:32]

Turns out it was failing coverage on a PermissionDenied check because we only have one user-modifiable hold type. Ran into this with the serializer too, and I just need to mock the HoldTypes.user_modifiable() method.

- Note taken on [2025-08-19 Tue 12:29]

This was branched from a branch that no longer exists and I need to rebase it.