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

📝 [#403] Document Notificaties API retry behavior #468

Merged

stevenbal merged 1 commit into master from issue/403-document-notification-retry-config

Oct 31, 2024

Contributor

stevenbal commented Oct 24, 2024

Fixes #403

Changes

Document Notificaties API retry behavior

stevenbal requested review from annashamray and Coperh

October 24, 2024 13:45

annashamray approved these changes

View reviewed changes

Collaborator

annashamray left a comment

👍

Coperh reviewed

View reviewed changes

docs/admin/notifications.rst Outdated Show resolved Hide resolved

Coperh reviewed

View reviewed changes

docs/admin/notifications.rst Outdated Show resolved Hide resolved

Coperh reviewed

View reviewed changes

docs/admin/notifications.rst Outdated Show resolved Hide resolved

Coperh reviewed

View reviewed changes

docs/admin/notifications.rst Show resolved Hide resolved

Coperh reviewed

View reviewed changes

docs/admin/notifications.rst Outdated Show resolved Hide resolved

Coperh requested changes

View reviewed changes

Contributor

Coperh left a comment

Looks good, just a few suggestions that makes it sound more correct and I think what the numbers t and backoff means should be stated.


          📝 [#403] Document Notificaties API retry behavior

38819b3

stevenbal force-pushed the issue/403-document-notification-retry-config branch from 3a05001 to 38819b3 Compare

October 31, 2024 09:21

stevenbal requested a review from Coperh

October 31, 2024 09:21

Coperh approved these changes

View reviewed changes

stevenbal merged commit 45fe648 into master

12 checks passed

stevenbal deleted the issue/403-document-notification-retry-config branch

October 31, 2024 11:14

danielmursa-dev added a commit that referenced this pull request


          [#468] Add identifier in TokenAuth and create migrations

c5cd151

danielmursa-dev added a commit that referenced this pull request


          [#468] Enable TokenAuthConfiguration

9cda2c1

danielmursa-dev added a commit that referenced this pull request


          [#468] Change namespaces for TokenAuthConfiguration

709b5c1

danielmursa-dev added a commit that referenced this pull request


          [#468] Add tests TokenAuthConfiguration

e871559

danielmursa-dev added a commit that referenced this pull request


          [#468] Add migrations test

5536da3

danielmursa-dev added a commit that referenced this pull request


          [#468] Update config_cli.rst

971c775

danielmursa-dev added a commit that referenced this pull request


          [#468] Improvements

d7a2dd7

danielmursa-dev added a commit that referenced this pull request


          [#468] Fix tests

3903e57

SonnyBA pushed a commit that referenced this pull request


          Merge pull request #468 from maykinmedia/issue/403-document-notificat…

a194581

…ion-retry-config

📝 [#403] Document Notificaties API retry behavior

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

Labels

None yet