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

Add push messaging (+ other assorted improvements) #124

Draft
wants to merge 428 commits into
base: development
Choose a base branch
from

Conversation

Rumperuu
Copy link
Contributor

@Rumperuu Rumperuu commented Nov 1, 2020

NB the Travis tests fail because it can't start the messaging service, but I think that will need someone with more Travis/Angular experience than me to fix.

This PR was originally intended to only add push notification functionality to the Web client. However, it has since expanded to into a bit of a monolith, which certainly isn't ideal and should be avoided in future, but I think I'm the only person working on this at the moment so it shouldn't cause any major issues.

This PR adds the following:

  • an updated, clarified project readme, along with security and contributing guidelines;
  • GitHub Issue templates;
  • assorted cosmetic improvements and bugfixes;
  • the ability for organisations to create topics;
  • the ability for customers to subscribe to and unsubscribe from topics;
  • the ability for organisations to push messages to customers; and
  • push notification rendering via the Notifications API.

See also Pear-Trading/LocalSpend-Tracker#12 & Pear-Trading/Foodloop-Server#140

Closes #123 #125 #127 #132 #133 #140 #141

@Rumperuu Rumperuu changed the title Push Notifications Add push messaging (+ other assorted improvements) Mar 20, 2021
@Rumperuu Rumperuu requested a review from TBSliver March 25, 2021 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment