Skip to content

Conversation

@lajp
Copy link
Member

@lajp lajp commented Oct 24, 2025

No description provided.

@lajp lajp requested a review from DrVilepis as a code owner October 24, 2025 07:37
lajp and others added 6 commits October 24, 2025 09:39
also remove all instances of impl IntoResponse in endpoint types
also thank you utoipa
@Eldemarkki
Copy link
Member

settings.toml.example is missing the new email fields

@Eldemarkki
Copy link
Member

Secured access mentions should be removed from error.rs and APISPEC.md

@Eldemarkki
Copy link
Member

The APISPEC seems to be out of date in general, for example it still has /auth/changepassword endpoint. Maybe the file is not needed now that we have OpenAPI, we could move the descriptions there?

@DrVilepis
Copy link
Member

The plan is to completely remove the apispec in favor of openapi. Also I doubt that the apispec is even 50% correct

Copy link
Member

@DrVilepis DrVilepis left a comment

Choose a reason for hiding this comment

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

My preliminary approval now that the pr is starting to be in its final state.

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.

4 participants