-
-
Notifications
You must be signed in to change notification settings - Fork 68
Open
Labels
enhancementNew feature or requestNew feature or requestepic:notificationsEpic: Notification AdaptersEpic: Notification Adaptersv2.0Target: v2.0 releaseTarget: v2.0 release
Description
User Story
As a trader, I want SMS notifications so that I receive urgent alerts on my phone.
Acceptance Criteria
-
SmsNotificationChannelimplementsINotificationChannel - Twilio API integration via
SmsOptions:- AccountSid (default from env: TWILIO_SID)
- AuthToken (default from env: TWILIO_TOKEN)
- FromNumber
- ToNumber
- Concise message format (within SMS limits)
- Handles Twilio rate limits
- Optional: Twilio package as soft dependency
Performance Metrics
| Metric | Target |
|---|---|
| Send latency | <10 seconds |
Dependencies
- [Epic 6.1] Define INotificationChannel interface
NuGet Dependencies
Twilio(optional, only needed if SMS used)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requestepic:notificationsEpic: Notification AdaptersEpic: Notification Adaptersv2.0Target: v2.0 releaseTarget: v2.0 release