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 notification #10

Open
strifel opened this issue Jul 5, 2019 · 6 comments
Open

Add push notification #10

strifel opened this issue Jul 5, 2019 · 6 comments
Labels
needs-api-implementation This issue needs (bigger) changes to the API

Comments

@strifel
Copy link
Member

strifel commented Jul 5, 2019

Add support for push notification.

Wouldn`t that be cool if you can receive a push notification when the Stundenpläne are online?

I think the recommended way is firebase.

@strifel strifel added the low priority These issues have a low priority label Jul 5, 2019
@strifel strifel self-assigned this Jul 5, 2019
@kagonlineteam kagonlineteam deleted a comment from robmroi Jul 5, 2019
@strifel
Copy link
Member Author

strifel commented Jul 5, 2019

To clarify this issue.
This is only meant as cloud notification.

One human sends Push Notification to all users.

For other ideas see #12 and #11.
Or create your own issue

@strifel
Copy link
Member Author

strifel commented Jul 6, 2019

After a bit more investigating. I think it is not possible (for us) to use firebase cloud messaging and own (on device) notifications.

We should probably either build up our own Notification System with our own API or cancel this issue.

So this issue is at least a lot postponed.

@strifel strifel added the needs-api-implementation This issue needs (bigger) changes to the API label Jul 6, 2019
@strifel
Copy link
Member Author

strifel commented Jul 6, 2019

This could theoretically work with #12 if we implement it in our API.

E.g.
Client -> "I want RPlan Updates" -> API
Client -> Opens Channel -> API
API -> "Here your Update" ->

This would be the best option I guess.
Maybe even for #11.
But I will implement #11 local based first.

For clarification:
This issue is (still) only for user messaging.
Like:
Client -> Opens Channel -> API
Admin -> Notification "Test" -> API
API -> Notification "Test" -> Client

@strifel strifel removed low priority These issues have a low priority needs-api-implementation This issue needs (bigger) changes to the API labels Mar 28, 2020
@strifel
Copy link
Member Author

strifel commented Mar 28, 2020

Wir machen das jetzt mit Firebase. Hierfür muss die Implementation erfolgen

@kagOT
Copy link
Member

kagOT commented Mar 28, 2020

Firebase

@strifel strifel added the needs-api-implementation This issue needs (bigger) changes to the API label May 4, 2020
@strifel strifel added this to the Release 1.1 milestone May 4, 2020
@strifel strifel removed this from the v1.1.0 milestone Oct 17, 2020
@strifel
Copy link
Member Author

strifel commented Oct 17, 2020

Hierfür wird noch auf die API gewartet. Grundsätzlich sind Notifications implementiert

@strifel strifel removed their assignment Oct 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-api-implementation This issue needs (bigger) changes to the API
Projects
None yet
Development

No branches or pull requests

2 participants