-
Notifications
You must be signed in to change notification settings - Fork 1.6k
fix(err): allow notifications to be edited #31043
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
Conversation
Hey @oliverb123! 👋 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
This PR enables editing of error tracking notification templates across multiple platforms and adds testing capabilities to the error tracking alert UI.
- Modified
/frontend/src/scenes/error-tracking/configuration/alerting/ErrorTrackingAlert.tsx
to enable testing configuration by settinghideTestingConfiguration: false
- Updated notification templates in Slack, Discord, and Microsoft Teams to make message content fields visible and editable by changing
hidden: False
in their respective template files - Modified
/frontend/src/scenes/pipeline/hogfunctions/hogFunctionConfigurationLogic.tsx
to conditionally exclude person data for error tracking notifications - Improved error tracking notification customization by exposing previously hidden fields in the UI for better user control
Greptile AI
5 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile
frontend/src/scenes/pipeline/hogfunctions/hogFunctionConfigurationLogic.tsx
Show resolved
Hide resolved
Size Change: +14 B (0%) Total Size: 13.2 MB ℹ️ View Unchanged
|
6e8dd91
to
888cf85
Compare
📸 UI snapshots have been updated2 snapshot changes in total. 0 added, 2 modified, 0 deleted:
Triggered by this commit. |
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
No description provided.