-
Notifications
You must be signed in to change notification settings - Fork 4
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
Notification System Not implemented #2
Comments
Consider support in in 0.9.0. https://docs.discourse.org/#tag/Notifications If you are interested in this project and like to contribute, maybe you could take a look at this. 🥰 @Lakr233 |
I'll take the job when I'm free, not too soon. And for APN, the notification support for these kind of project requires either two way:
Which one you prefer? |
Both seem fine for me. I haven’t written much APN code before. Does the first method require a external server to support it? I prefer a solution that does not rely on an external server.
When creating a user-api-key from a discourse server, we can add push_url as a parameter.
See more info here https://meta.discourse.org/t/user-api-keys-specification/48536 |
Thanks. I've assigned this issue to you. Feel free to cancel the assignment if you have other plan. |
Web Hook requires an application to extract information from the hook, and translate it to APN endpoint. So an external server is required. Talked with author of Mastoot, he host a server jsut for that. |
Then I think the second one is our best choice. 😀 |
No description provided.
The text was updated successfully, but these errors were encountered: