[Due for payment 2025-06-05] [Due for payment 2025-06-04] [Due for payment 2025-06-02] [Simplified Actions] [App] Implement DeleteAppReport

[luacmartins]

Implement this section of the #migrate: Simplified actions framework for all reports design doc

Issue Owner

Current Issue Owner: @kadiealexander

[melvin-bot]

Triggered auto assignment to @kadiealexander (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details. Please add this Feature request to a GH project, as outlined in the SO.

[luacmartins]

@ikevin127 you can start working on implementing this one! Please prioritize it since it's part of a critical project.

[ikevin127]

Will start working on this today 👍

[luacmartins]

Nice! Let me know if you have questions. Right now we only have the DeleteReport command (same param signature), the PR to add the DeleteAppReport alias is in review

[ikevin127]

Sure! As far as I can tell from OP section, the DeleteAppReport implementation does not involve any new design changes other than the copy variation for the new API call:

Delete report
Are you sure you want to delete this report?

(already exists currently [DeleteMoneyRequest]):

Delete expense
Are you sure you want to delete this expense?

The goal here being that in case a report has more expenses we will use the new DeleteAppReport that will optimistically delete the report and unreport all transactions on the report, which current DeleteMoneyRequest does not do.

Did I understand this task correctly ?

the PR to add the DeleteAppReport alias is in review

Does this mean that if I call the API right now to test the implementation, it would not work yet since the backend PR wasn't merged right ? If that's the case please let me know as soon as the BE part is available (staging or prod).