[Recovery] Get status of email registration

safe-global / safe-wallet-web

Safe{Wallet} – multisig EVM wallet

https://app.safe.global

GNU General Public License v3.0

355 stars 420 forks source link

[Recovery] Get status of email registration #3291

Open usame-algan opened 8 months ago

usame-algan commented 8 months ago

Requirements

[x] Show a new "Email notification" section on the settings page (Figma)
[x] Clicking the "Sign to view" button should fetch the GET endpoint of the CGW
[x] If there is an email registered it should be displayed (Figma)
[x] If there is no email registered there should be a "No email found" message

usame-algan commented 8 months ago

This can be tested here https://github.com/safe-global/safe-wallet-web/pull/3331

liliya-soroka commented 8 months ago

@usame-algan , should " Clicking the "Sign to view" button should fetch the GET endpoint of the CGW " be available for non-owner? In the latest version , I can sign as non-owner and see the email status

usame-algan commented 8 months ago

Good point! This will be added with https://github.com/safe-global/safe-wallet-web/issues/3320 so only owners can press that button.

liliya-soroka commented 8 months ago

Verified. Note: state for the owner with and without signing will be checked together with https://github.com/safe-global/safe-wallet-web/issues/3320