Skip to content
This repository has been archived by the owner on Sep 21, 2020. It is now read-only.

Implement simple notifications #49

Closed
thebitguru opened this issue Apr 8, 2020 · 4 comments
Closed

Implement simple notifications #49

thebitguru opened this issue Apr 8, 2020 · 4 comments
Assignees
@prescottprue @MrSimsek

Comments

@thebitguru
Copy link
Contributor

thebitguru commented Apr 8, 2020 •
edited
Loading

Please see details in the design document that I shared. I will update this issue with the details in a bit.
@thebitguru thebitguru assigned thebitguru and unassigned thebitguru Apr 8, 2020
@MrSimsek
Copy link
Contributor

MrSimsek commented Apr 8, 2020

Hey

@thebitguru thebitguru mentioned this issue Apr 8, 2020
Open
@prescottprue prescottprue mentioned this issue Apr 12, 2020
Merged
9 tasks
@prescottprue prescottprue self-assigned this Apr 12, 2020
@prescottprue
Copy link
Collaborator

Have a used FCM a bunch before, so started working on this after chatting with @thebitguru. The work is currently in the next branch PR for the version update

@prescottprue prescottprue mentioned this issue Apr 12, 2020
Merged
@prescottprue
Copy link
Collaborator

@thebitguru Did you want to close this ticket since we got the simple setup in place for new requests (email + FCM), and the daily (email)? Or did you want to implement some more functionality first?

@thebitguru
Copy link
Contributor Author

Yes, I am closing this issue and will create another one for Milestone 5 with a few more changes to get it ready for general rollout.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@prescottprue prescottprue

@MrSimsek MrSimsek

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@thebitguru @prescottprue @MrSimsek