Skip to content

Replace AlertsController with trame-alerts#80

Merged
PaulHax merged 3 commits intomainfrom
alerts
Feb 9, 2026
Merged

Replace AlertsController with trame-alerts#80
PaulHax merged 3 commits intomainfrom
alerts

Conversation

@PaulHax
Copy link
Contributor

@PaulHax PaulHax commented Feb 9, 2026

Summary

  • Replaces custom AlertsController with the trame-alerts library for stacked notification support
  • Uses get_alerts_service(server) instead of injecting a controller dependency
  • Swaps manual VSnackbar template in UI with AlertsProvider + AlertsPopup components

Test plan

  • Trigger a decision and verify loading/completion alerts appear and dismiss
  • Import experiments and verify loading/completion alerts appear and dismiss
  • Verify alerts stack when multiple are active
  • Verify alerts appear in bottom-right corner

@PaulHax PaulHax merged commit 2727a79 into main Feb 9, 2026
6 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

Comments