Skip to content

Conversation

@yashsinghcodes
Copy link
Member

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds severity and origin tracking to the notification system by introducing two new string fields to the Notification struct. These fields enable better categorization and filtering of notifications based on their importance level and source.

Key changes:

  • Added Severity and Origin fields to the Notification struct with corresponding JSON and datastore tags
  • Updated CreateOrgNotification function signature to accept severity and origin parameters
  • Updated all call sites (15+) across the codebase to provide severity levels (LOW, MEDIUM, HIGH, CRITICAL) and origin identifiers

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
structs.go Added Severity and Origin string fields to the Notification struct
notifications.go Updated CreateOrgNotification function signature to include severity and origin parameters, and initialized these fields in notification objects
shared.go Updated 10 CreateOrgNotification calls with severity levels (MEDIUM, CRITICAL, HIGH) and origin identifiers for various workflow and authentication scenarios; also uncommented a warning log for missing API keys
oauth2.go Added HIGH severity and "oauth" origin to OAuth2 error notification
db-connector.go Added MEDIUM severity for datastore automation errors and workflow execution time notifications
ai.go Added LOW severity and "ai" origin for AI-generated app notifications

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@yashsinghcodes yashsinghcodes merged commit 51924cd into Shuffle:main Nov 24, 2025
7 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant