safe-global / safe-client-gateway / 7249845280 / 1

89%

main: 89%

push

github

feat: add notifications for recovery proposals (#922)

This adds base notifications logic for alerts of owner management transactions, be them "direct" with the Safe or via in a batched `multiSend` call:

- `addOwnerWithThreshold`
- `removeOwner`
- `swapOwner`
- `changeThreshold`

If an alert is received containing one of the above, it is forwarded onto the email provider with what the new Safe state would be after the proposed recovery attempt is executed.

* feat: add notifications for recovery proposals

* feat: add notifications for recovery proposals

* fix: add test for `addOwnerWithThreshold`

* refactor: address review + add test comment

* chore: add todo tests

* fix: add test coverage

* refactor: only call `_notifyUnknownTransaction` once

* Revert "refactor: only call `_notifyUnknownTransaction` once"

This reverts commit 110716087.

* fix: retrieve email addresses first

* fix: change status code in tests

* fix: move `return`

204 of 1646 branches covered (0.0%)

Branch coverage included in aggregate %.

6258 of 10114 relevant lines covered (61.87%)

4.64 hits per line