-
Notifications
You must be signed in to change notification settings - Fork 33
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
Document Notification API V2 #202
Comments
/notifications/<not_UUID>/respond |
Both V1 and V2 share |
Is there a new websocket endpoint as well? |
@Rexios80 No but it sends new kinds of objects down the websocket IIRC :-) |
Are there equivalent objects just with different type strings? |
What? The Notification objects are completely different. In the Websocket it is indentified by a different type. |
I meant equivalent as in they represent the same event. The answer should be yes. |
So if I understood it correctly... only the group events are new in the websocket? Everything else is the same? |
How do we want to handle this? I see a few options:
If we do decide to keep the old notification APIs around, should we deprecate them? |
Huh? Just because there is a V2 doesn't mean V1 is deprecated. Notifications from both V1 and V2 share namespace (a.k.a |
Old API: /auth/user/notifications
New API: /notifications
Objects and everything are COMPLETELY different. This is a complete rewrite.
The text was updated successfully, but these errors were encountered: