Skip to content
Snippets Groups Projects

Generalise notifications and implement SMS notifications

Merged Nik | Klampfradler requested to merge 165-generalise-notifications into master

This MR implements some generalisations for the notification system.

  1. Use more generic naming in model
  2. Create utility library ofr sending notifications
  3. Implement SMS channel (Twilio) in addition to e-mail channel
  4. Make channels configurable in admin backend

Special attention has been paid to:

  • Allow easy addition of channels
  • Ensure only configured channels can be selected
  • Ensure channels that have been selected, but have becom unavailable, are not tried
>>> from aleksis.core.models import Person, Notification
>>> nik = Person.objects.first()
>>> noti = Notification.objects.create(title="Tag des Raclettes", description="Bitte überbacken Sie Ihre Vertretungsaufgaben umgehend mit Käse 🧀!", link="http://www.raclette.de/shop/", recipient=nik, sender="Feiertage-App")

IMG_20200122_232155

Closes #165 (closed)

Edited by Nik | Klampfradler

Merge request reports

Pipeline #720 failed

Pipeline failed for 413582bb on 165-generalise-notifications

Approval is optional

Merged by Jonathan WethJonathan Weth 5 years ago (Jan 23, 2020 10:17am UTC)

Merge details

Pipeline #722 failed

Pipeline failed for 0d60c9f7 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading