Skip to content

Conversation

@nickvergessen
Copy link
Member

No description provided.

@nickvergessen nickvergessen force-pushed the fair-use-push branch 2 times, most recently from af8feb7 to a90f100 Compare October 22, 2021 08:58
@nickvergessen nickvergessen added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Oct 22, 2021
@nickvergessen nickvergessen marked this pull request as ready for review October 22, 2021 09:02
@nickvergessen nickvergessen requested review from a team, ArtificialOwl, icewind1991, skjnldsv and vitormattos and removed request for a team October 22, 2021 09:02
Copy link
Member

@blizzz blizzz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, but a remark.

@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Oct 22, 2021
nickvergessen and others added 4 commits October 23, 2021 00:54
We want to keep offering our push notification service for free, but large
users overload our infrastructure. For this reason we have to rate-limit the
use of push notifications. If you need this feature, consider setting up your
own push server or using Nextcloud Enterprise.

Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Vitor Mattos <vitor@php.rio>
Signed-off-by: Vitor Mattos <vitor@php.rio>
Signed-off-by: Vitor Mattos <vitor@php.rio>
@nickvergessen nickvergessen requested a review from kesselb October 22, 2021 22:55
@skjnldsv skjnldsv merged commit a06001e into master Oct 23, 2021
@skjnldsv skjnldsv deleted the fair-use-push branch October 23, 2021 08:53
@skjnldsv skjnldsv mentioned this pull request Oct 25, 2021
25 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish enhancement feature: activity and notification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants