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.

Sign up for GitHub

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

Jump to bottom

Notifications API #337

Closed
MorrisJobke opened this issue Jan 26, 2017 · 6 comments
Closed

Notifications API #337

MorrisJobke opened this issue Jan 26, 2017 · 6 comments
Assignees
@nickvergessen
Labels
3. to review enhancement high

Comments

@MorrisJobke
Copy link
Member

@nickvergessen Please then also document the notifications API from nextcloud/notifications#59
@nickvergessen
Copy link
Member

Docs are going to be and already are in the PR

@MorrisJobke
Copy link
Member Author

Docs are going to be and already are in the PR

I guess we then need a page in the docs where we link all those stray documentation about API for different apps.

@MorrisJobke
Copy link
Member Author

🙈

@nickvergessen nickvergessen self-assigned this Jan 30, 2017
@MorrisJobke
Copy link
Member Author

@marinofaggiana The API is documented already in the linked PR - see https://github.com/nextcloud/notifications/blob/push-notification/docs/push-v3.md

@marinofaggiana
Copy link
Member

yes yes this is a good point 👍

@MorrisJobke MorrisJobke mentioned this issue Feb 24, 2017
Merged
@MorrisJobke
Copy link
Member Author

Fix in #365

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

@nickvergessen nickvergessen

Labels
3. to review enhancement high
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@nickvergessen @MorrisJobke @marinofaggiana