Skip to content

Alert-system: Implement GUID#2645

Merged
sudip-khanal merged 4 commits intoproject/alert-system-workflowfrom
feat/implement-guid
Feb 6, 2026
Merged

Alert-system: Implement GUID#2645
sudip-khanal merged 4 commits intoproject/alert-system-workflowfrom
feat/implement-guid

Conversation

@sandeshit
Copy link
Contributor

@sandeshit sandeshit commented Feb 2, 2026

Changes

  • Implement GUID.
  • Add forecasted field config.
  • Remove correlation_id from alert notification flow
  • Use parent_guid for alert notification tracking

Checklist

Things that should succeed before merging.

  • Updated/ran unit tests
  • Updated CHANGELOG.md

Release

If there is a version update, make sure to tag the repository with the latest version.

@sandeshit sandeshit force-pushed the feat/implement-guid branch from 995704d to 4dc1b8d Compare February 3, 2026 08:40
@sudip-khanal sudip-khanal marked this pull request as draft February 4, 2026 04:45
@sandeshit sandeshit requested a review from susilnem February 5, 2026 03:56
@sudip-khanal sudip-khanal force-pushed the project/alert-system-workflow branch 2 times, most recently from 14f9d39 to d8f3fe3 Compare February 6, 2026 06:02
sandeshit and others added 4 commits February 6, 2026 12:34
- Replace correlation function with guid.
- Add forecasted filter.
- Add guid and parent_guid field in models.
- Squash migrations.
- Remove correlation_id from alert notification flow
- Use parent_guid for alert notification tracking
- Disable parallel processing to keep execution deterministic
- Move email processing logic out of tasks.py
@sudip-khanal sudip-khanal marked this pull request as ready for review February 6, 2026 07:53
@sudip-khanal sudip-khanal merged commit afe5461 into project/alert-system-workflow Feb 6, 2026
3 checks passed
@sudip-khanal sudip-khanal deleted the feat/implement-guid branch February 6, 2026 07:54
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.

3 participants