safe-wallet-web icon indicating copy to clipboard operation
safe-wallet-web copied to clipboard

[Recovery] Get status of email registration

Open usame-algan opened this issue 1 year ago • 4 comments

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 avatar Feb 20 '24 11:02 usame-algan

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

usame-algan avatar Feb 26 '24 10:02 usame-algan

@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

liliya-soroka avatar Feb 26 '24 12:02 liliya-soroka

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

usame-algan avatar Feb 26 '24 14:02 usame-algan

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

liliya-soroka avatar Feb 26 '24 15:02 liliya-soroka