Skip to content

updated sunset references - #184

Merged
arhimede merged 3 commits into
mainfrom
sunset
Oct 20, 2026
Merged

arhimede merged 3 commits into
mainfrom
sunset

Conversation

@bidi47

@bidi47 bidi47 commented Oct 19, 2026

Copy link
Copy Markdown
Member

related to dotkernel/api#556

Signed-off-by: bidi47 <bidi@apidemia.com>
| `ConflictException` | `409 Conflict` | `UserService`, `AdminService`, `DeprecationMiddleware`, `ResourceProviderMiddleware` |
| `ExpiredException` | `410 Gone` | the password reset handlers |
| `ConflictException` | `409 Conflict` | `UserService`, `AdminService`, `ResourceProviderMiddleware` |
| `ExpiredException` | `410 Gone` | the password reset handlers, `DeprecationMiddleware` once a `sunset` date is reached |

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should we mark in some way that this behaviour
was instroduced in a certain PR , in the 7.2 release, at a certain date ?

@alexmerlin I am note sure what solution we have here to mark a difference between 2 releases ?

should we create in the menu : instead of 7.0 documentation, to have somethjig like : 7.1, 7.2 ?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

add a note: introduced in 7.2 , and the behaviour in 7.1 was other ways

@bidi47 bidi47 Oct 20, 2026 •

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

reviewed the docs and added references to different behavior between the recent api versions (v7, v7.1, v7.2), where applicable

note: i considered api v7.2 as released, so we should consider finishing the 7.2 milestone and releasing it before this PR

Signed-off-by: bidi47 <bidi@apidemia.com>
Signed-off-by: bidi47 <bidi@apidemia.com>
@bidi47
bidi47 requested a review from arhimede October 20, 2026 15:25
@arhimede
arhimede merged commit a5256f8 into main Oct 20, 2026
10 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.

2 participants