Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Break out notification-styling into type-dependent reusable components #108

Open
hmpf opened this issue Aug 7, 2020 · 1 comment
Open
Labels
enhancement New feature or request notification Affects the notification system

Comments

@hmpf
Copy link
Contributor

hmpf commented Aug 7, 2020

There are multiple ways of sending SMSes: SaaSes like Twilio, talking to a locally connected phone etc. The style of the content is the same however, 160 characters. That styling can possibly be reused for slack.

For email, Django has multiple backends, so we hopefully can use that instead of our own plugin system. But: the styling for subject and body can be reused for Teams channels.

@hmpf hmpf added enhancement New feature or request priority: low labels Aug 7, 2020
@hmpf hmpf changed the title Break out styling into type-dependent reusable components Break out notification-styling into type-dependent reusable components Aug 19, 2020
@hmpf
Copy link
Contributor Author

hmpf commented Sep 15, 2020

See #121.
Also, this is a requirement for #107

@hmpf hmpf added this to the blue sky milestone Sep 29, 2020
@hmpf hmpf added the notification Affects the notification system label Sep 29, 2020
@hmpf hmpf modified the milestones: Blue sky, New notification system Oct 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request notification Affects the notification system
Projects
None yet
Development

No branches or pull requests

2 participants