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

WIP - Documentation api, addressing warnings #220

Open
wants to merge 14 commits into
base: dev
Choose a base branch
from

Conversation

AlbertoCasasOrtiz
Copy link
Collaborator

@AlbertoCasasOrtiz AlbertoCasasOrtiz commented Sep 24, 2024

@carmichaelong This PR is bigger than I expected, so I will take some time to check everything is correct. I basically reformatted everything to remove warnings and errors, and added swagger documentation and docstrings to views.py.

It will probably collide with dev, or any changes we have pending, so we should address this PR with caution.

Copy link
Collaborator Author

@AlbertoCasasOrtiz AlbertoCasasOrtiz left a comment

Choose a reason for hiding this comment

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

I have just fixed conflicts and reviewed this PR and everything looks good. The only thing is that the pycharm formatter did some silly things, but i will fix that.

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.

1 participant