-
Notifications
You must be signed in to change notification settings - Fork 0
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
docs: OpenApi Documentation #9
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice advance 🚀, however, check the review comments and note that the following endpoints are missing:
- GET endpoint to list the files in a given location ✅ (See ListFileReq.java).
- GET endpoint to download the file with the given UUID ✅ (See DownloadFileReq.java).
- POST/PUT endpoint to share the given file with the given user ✅ (See ShareWithReq.java).
- POST/PUT/DELETE endpoint to un-share the given file with the given user ✅ (See UnShareWithReq.java).
- GET endpoint to see the list of users the given file is shared to ✅ (See SharedWithWhoReq.java).
@shoriwe Hi 👋🏽, is the |
You should now have write access to this repository too |
1b30258
to
d76445f
Compare
@PedroChaparro Thanks for checking, I already made new changes, please could you check it? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work python team 🐍
* docs: OpenApi Specification (#9) * chore: Update docker-compose (#12) * chore: Add missing env variables and services to docker-compose * chore: Use worker default env variables * ci: Linter and formatter (#14) * chore: Add recommended vscode extensions * chore(deps): Install black formatter and ruff linter * style: Lint and format files * docs: Add linting and formatting commands to README * ci: Add linter and style checks to integration pipeline * feat: endpoint login (#15) * feat: endpoint challenge (#33) --------- Co-authored-by: Antonio Donis <[email protected]> Co-authored-by: Pedro Andrés Chaparro Quintero <[email protected]>
Include: