PySyft icon indicating copy to clipboard operation
PySyft copied to clipboard

Enable notifications via settings api

Open IonesioJunior opened this issue 1 year ago • 0 comments

Using the new Notifier Service, the node admin must be able to enable/disable the node notifier.

But to achieve that, there are some edge cases to keep in mind.

  • [X] By UX/UI reasons, the api used to enable / disable it will be set at client.settints.
    • To enable: client.settings.enable_notifications(token=<TOKEN>)
    • To disable: client.settings.disable_notifications()
  • [X] If email token isn't provided, returns a SyftError informing that email_token parameter is mandatory.
  • [X] If a regular user tries to use this API, it must receive the regular PySyft non authorized error.
  • [X] If admin provides all the informations properly, it will return a SyftSuccess message.

IonesioJunior avatar Feb 16 '24 22:02 IonesioJunior