Skip to content
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

Remove Swagger UI #973

Merged
merged 2 commits into from
Dec 11, 2024
Merged

Remove Swagger UI #973

merged 2 commits into from
Dec 11, 2024

Conversation

tnotheis
Copy link
Member

Readiness checklist

  • I added/updated unit tests.
  • I added/updated integration tests.
  • I ensured that the PR title is good enough for the changelog.
  • I labeled the PR.

The Swagger UI isn't usable anyway, because of some problems with the models. So we might as well remove it in order to not cause any confusion.

@tnotheis tnotheis added the enhancement New feature or request label Dec 11, 2024
@tnotheis tnotheis self-assigned this Dec 11, 2024
@tnotheis tnotheis merged commit cd2c83c into main Dec 11, 2024
24 of 26 checks passed
@tnotheis tnotheis deleted the remove-swaggerui-authentication branch December 11, 2024 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MH321Productions MH321Productions MH321Productions approved these changes

Assignees

@tnotheis tnotheis

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tnotheis @MH321Productions