feat: add deleteAfterResult option to approval controller

[matthewwalsh0]

Explanation

Currently, when accepting an approval request with the ApprovalController, the request is removed from the state as soon as accept is called.

This adds an optional deleteAfterResult option which (if waitForResult is also set) retains the approval request in the state until a result callback is invoked by the creator of the approval request.

Since the clients typically route based on the presence of an approval request in the state, this can simplify some scenarios where the route or component is needed until all the approval request related processing is complete.

This also aligns with the current ApprovalController patch in the mobile client.

References

Changelog

@metamask/approval-controller

• ADDED: Add deleteAfterResult option to accept method to delay removal of approval request from state.

Checklist

• I've updated the test suite for new or updated code as appropriate
• I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
• I've highlighted breaking changes using the "BREAKING" category above as appropriate