Skip to content

Send initial test notification when users subscribe to push notifications

Jano requested to merge send_test_push_notification into master

What does this MR do?

This MR introduces a test/greeting notification to be sent right after users activate push notifications. This will not only simplify testing push notifications with different devices, it will also demonstrate to users what push notifications are and prove that they work.

This MR also includes some refactorings

How confident are you it won't break things if deployed?

Very. I tested sending, receiving, subscribing and unsubscrubing from push notifications and I can not imagine what else should break.

How to test

  1. Checkout branch locally
  2. Login as foodsaver
  3. Activate push notifications
  4. Receive a test notification

Checklist

  • added a test, or explain why one is not needed/possible...
  • no unrelated changes
  • asked someone for a code review
  • joined #foodsharing-beta channel at https://slackin.yunity.org
  • added an entry to CHANGELOG.md (description, merge request link, username(s))
  • Once your MR has been merged, you are responsible to update the #foodsharing-beta Slack channel about what has been changed here. They will test your work in different browsers, roles or other settings

Merge request reports