Skip to content

beta to stable - #130

Merged
Sekiro-kost merged 3 commits into
masterfrom
beta
Nov 3, 2026
Merged

Sekiro-kost merged 3 commits into
masterfrom
beta

Conversation

@Sekiro-kost

Copy link
Copy Markdown
Contributor

Description

Suggested changelog entry

Related issues/external references

Fixes #

Types of changes

  • Bug fix (non-breaking change which fixes)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
    • This change is only breaking for integrators, not for external standards or end-users.
  • Documentation improvement

PR checklist

  • I have checked there is no other PR open for the same change.
  • I have read the Contribution Guidelines.
  • I grant the project the right to include and distribute the code under the GNU.
  • I have added tests to cover my changes.
  • I have verified that the code complies with the projects coding standards.
  • [Required for new sniffs] I have added MD documentation for the sniff.

<!--

Please make sure your pull request passes all continuous integration checks!

PRs which are failing their CI checks will likely be ignored by the maintainers.

PRs using atomic, descriptive commits are hugely appreciated as it will make
reviewing your changes easier for the maintainers.

-->

… resend them via cron

  Notifications refused with "Quotas exceeded" were dropped after a useless retry,
  so they appeared in the app history but never reached the phone.

  - skip the immediate retry on quota errors (it consumed quota for nothing)
  - store refused notifications in core/data/fcmQueue.json (flock-protected)
  - resend them in order from a new cron(), stopping when the quota is hit again
  - queue new notifications behind pending ones to preserve ordering
  - drop expired items: 1h max, or the ask timeout (remaining timeout is recomputed)
  - remove obsolete quota i18n string, add en_US translations, update changelog
@Sekiro-kost
Sekiro-kost merged commit c6f2c94 into master Nov 3, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant