Clean up code assuming a money request is P2P when policy isn't set

[cead22]

• Coming from this GH comment Client-side violations for money request updates #34402 (comment) and this slack thread https://expensify.slack.com/archives/C01GTK53T8Q/p1707341460136359?thread_ts=1707339619.099769&cid=C01GTK53T8Q
• The code we're using to decide whether a money request is a workspace request or a P2P request relies on policy being set
• This works when creating money request because the methods we call don't pass a policyID
• But P2P request get put on a report that's on a personal workspace, the the policy/policyID is set in the code used for updating money request
• Let's update the code for updating money requests to use PolicyUtils.isPaidGroupPolicy to determine if the request is a workspace request or a P2P request

@lindboe please comment for assignment

[lindboe]

Commenting for assignment

[cead22]

@cdanwards please comment for assignment

[cdanwards]

Commenting for assignment

[melvin-bot]

This issue has not been updated in over 15 days. @cead22, @lindboe, @cdanwards 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!