Skip to content

feat: add notification priority levels#146

Merged
Abd-Standard merged 2 commits into
Core-Foundry:mainfrom
CleanDev-Fix:feat/notification-priority-levels-130
Jun 22, 2026
Merged

feat: add notification priority levels#146
Abd-Standard merged 2 commits into
Core-Foundry:mainfrom
CleanDev-Fix:feat/notification-priority-levels-130

Conversation

@CleanDev-Fix

Copy link
Copy Markdown
Contributor

Summary

Adds notification priority metadata to the AutoShare contract events and stored group details.

Changes

  • Added a NotificationPriority enum with Low, Standard, High, and Critical levels.
  • Stored the default group priority on AutoShareDetails.
  • Published priority metadata on contract notification events while keeping category as the trailing topic for existing category filters.
  • Expanded notification tests to cover persisted priority, emitted priority topics, and event-shape compatibility.

Testing/Verification

  • docker run --rm -v "<repo>:/work" -w /work/contract/contracts/hello-world rust:1.86 cargo test passed.
  • git diff --check passed.
  • cargo fmt -- --check could not be run in this hot-runner shell because host Cargo is unavailable and the tested Rust container lacks cargo-fmt/rustup.

Related issue: closes #130

@Abd-Standard Abd-Standard merged commit 78b9fdd into Core-Foundry:main Jun 22, 2026
1 of 3 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.

[Contracts] Add Notification Priority Levels

2 participants