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.

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

Swagger Docs #126

Merged
merged 3 commits into from
May 29, 2024
Merged

Swagger Docs #126

merged 3 commits into from
May 29, 2024

Conversation

sam57719
Copy link
Collaborator

/docs endpoint now pulls in smib/webserver/README.md as its description

Refactored smib/webserver/main.py slightly, grouping the /smib/events path endpoints together so they share common settings for the swagger docs

…gets pulled into the swagger docs (/docs endpoint)

Update dependencies, add version retrieval function, and README.md rendering

This commit updates the 'dataclasses-json' package and removes an unnecessary PyYAML wheel file from 'poetry.lock'. It also introduces 'get_version()' in 'utils.py' and modifies 'smib-fast.Dockerfile' to enable the root package installation with Poetry. Additionally, it includes a change to render the 'smib/webserver/README.md' file in the FastAPI app.
Refactored the webserver main to extract logic into separate functions for better code maintainability and readability. Also updated the README.md file of the webserver to contain more detailed information about the APIs, the server, and its endpoints, and improved error handling with specific responses for certain status codes.
@sam57719 sam57719 linked an issue May 28, 2024 that may be closed by this pull request
Copy link
Member

@sjefferson99 sjefferson99 left a comment

Choose a reason for hiding this comment

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

Good.

@sjefferson99 sjefferson99 merged commit 6348619 into develop May 29, 2024
@sjefferson99 sjefferson99 deleted the 9-swagger-docs branch May 29, 2024 19:42
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.

Add/improve Swagger docs in fastapi
2 participants