unikubehq / cli

The convenient command line interface (CLI) to the unikube platform. :computer:
https://unikube.io
Apache License 2.0
18 stars 1 forks source link

Sentry Debugging #47

Open SteinRobert opened 3 years ago

SteinRobert commented 3 years ago

It would help to add Sentry to the CLI for future debugging purposes.

However, probably not every user wants the CLI to send Sentry reports - for privacy reasons. So we need some kind of opt-in / opt-out mechanism which enables/disables Sentry for every following command.

For the future we could add more detailed options like - Sentry Reports enabled / Sentry Reports without OS information / No Sentry Reports etc.

buschNT commented 3 years ago

@SteinRobert

To integrate sentry we would need to introduce the required user settings as mentioned. This, I guess, should be added to the user service, e.g. (GraphQL):

{
  user {
    id
    email
    name
    settings {
      group
      name
      value
    }
  }
}

Do we already have user settings exposed in GraphQL? I can't find anything. However, where are the notification settings in the frontend. I just wanted to ask, because to start this feature at the cli, we first have to introduce the settings in the user service.

SteinRobert commented 3 years ago

The user service is not very sophisticated at the moment. These settings are not available. However, I feel like the setting should be stored locally. One should be able to enable and disable this setting completely offline, no depend on the internet connection.