Skip to content

[stable34] fix(web-push): improve fetching notifications with web-push - #3551

Merged
Antreesy merged 5 commits into
stable34from
backport/3549/stable34
Oct 29, 2026
Merged

Antreesy merged 5 commits into
stable34from
backport/3549/stable34

Conversation

@backportbot

@backportbot backportbot Bot commented Oct 29, 2026

Copy link
Copy Markdown

Backport of PR #3549

Web push activation set hasNotifyPush, so every later web push
skipped the fetch.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
The server sends the activation push only for a new subscription.
On later page loads onActivated was never called, so polling stayed
at the default interval.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
If the app mounts after the load event, the listener never fired.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy

Copy link
Copy Markdown
Collaborator

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@Antreesy
Antreesy merged commit fe39f33 into stable34 Oct 29, 2026
34 checks passed
@Antreesy
Antreesy deleted the backport/3549/stable34 branch October 29, 2026 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants