Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sentry Debugging #47

Open
SteinRobert opened this issue Jun 17, 2021 · 2 comments
Open

Sentry Debugging #47

SteinRobert opened this issue Jun 17, 2021 · 2 comments
Labels
1-low enhancement New feature or request

Comments

@SteinRobert
Copy link
Contributor

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.

@SteinRobert SteinRobert added the enhancement New feature or request label Jun 17, 2021
@Schille Schille added this to the Version 1.1 milestone Aug 25, 2021
@buschNT
Copy link
Collaborator

buschNT commented Sep 30, 2021

@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
Copy link
Contributor Author

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.

@Schille Schille removed this from the Version 1.1 milestone Oct 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1-low enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants