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.

Sign up for GitHub

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

Jump to bottom

Implement Sentry error logging for GUI #1153

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

Implement Sentry error logging for GUI #1153

wants to merge 8 commits into from

Conversation

ImUrX
Copy link
Member

@ImUrX ImUrX commented Sep 3, 2024 •
edited
Loading

Closes #1049, #393

@ImUrX ImUrX added Area: Documentation Improvements or additions to documentation Area: GUI Related to the GUI Type: Enhancement Adds or improves a feature labels Sep 3, 2024
@ImUrX ImUrX linked an issue Sep 3, 2024 that may be closed by this pull request
Add sentry.io error tracking #393
Open
@TheDevMinerTV
Copy link
Member

I'd rename Data collection to Error Tracking. I'd also remove the trace targets as we don't have anything that actually accepts the distributed traces.

@ImUrX
Copy link
Member Author

ImUrX commented Sep 5, 2024

yeah you are right on the trace targets part, the data collection yeah should also be renamed to error tracking

@github-actions github-actions bot added Area: Continuous Integration Automated testing and deployment Area: Translation Improvements or additions to translations labels Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Erimelowo Erimelowo Awaiting requested review from Erimelowo Erimelowo will be requested when the pull request is marked ready for review Erimelowo is a code owner

@loucass003 loucass003 Awaiting requested review from loucass003 loucass003 will be requested when the pull request is marked ready for review loucass003 is a code owner

@Eirenliel Eirenliel Awaiting requested review from Eirenliel Eirenliel will be requested when the pull request is marked ready for review Eirenliel is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
Area: Continuous Integration Automated testing and deployment Area: Documentation Improvements or additions to documentation Area: GUI Related to the GUI Area: Translation Improvements or additions to translations Type: Enhancement Adds or improves a feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create Behavior category under the Interface settings Add sentry.io error tracking
2 participants
@ImUrX @TheDevMinerTV