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

Add Rswag, spec the users api controller #120

Merged
merged 4 commits into from
Jun 24, 2022
Merged

Add Rswag, spec the users api controller #120

merged 4 commits into from
Jun 24, 2022

Conversation

TomNaessens
Copy link
Member

@TomNaessens TomNaessens commented Jun 5, 2022 •
edited
Loading

Fixes #90

Uses RSwag to not only create an API documentation, but creates specs at the same time.

It's possible this will require a few changes when deploying, we'll see when this is merged.

Result:

localhost_3000_api-docs_index html

@TomNaessens TomNaessens force-pushed the feature/90/document-api-routes branch from 96ab065 to 0da4e19 Compare June 5, 2022 16:05
@TomNaessens TomNaessens marked this pull request as draft June 5, 2022 16:05
@TomNaessens TomNaessens force-pushed the feature/90/document-api-routes branch 2 times, most recently from 63463ad to 18af07f Compare June 5, 2022 16:49
@TomNaessens TomNaessens force-pushed the feature/90/document-api-routes branch 3 times, most recently from 95b0207 to 01028ed Compare June 12, 2022 12:25
@TomNaessens TomNaessens requested a review from redfast00 June 12, 2022 15:48
@TomNaessens TomNaessens marked this pull request as ready for review June 12, 2022 15:48
@TomNaessens TomNaessens force-pushed the feature/90/document-api-routes branch from 2119610 to 5fdad2f Compare June 12, 2022 15:50
@TomNaessens TomNaessens changed the title Draft: Add Rswag, spec the users api controller Add Rswag, spec the users api controller Jun 12, 2022
@TomNaessens TomNaessens mentioned this pull request Jun 12, 2022
Closed
6 tasks
@TomNaessens TomNaessens force-pushed the feature/90/document-api-routes branch from 5fdad2f to 4a86f84 Compare June 24, 2022 16:22
@TomNaessens TomNaessens merged commit 5f2ad5e into master Jun 24, 2022
@TomNaessens TomNaessens deleted the feature/90/document-api-routes branch June 24, 2022 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@redfast00 redfast00 Awaiting requested review from redfast00

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Document API routes
1 participant
@TomNaessens