-
Notifications
You must be signed in to change notification settings - Fork 399
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
feat(api): add the new main license for the upload #2462
Conversation
92e9b1c
to
1da2768
Compare
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.
Minor changes needed.
0dd843d
to
575d2dd
Compare
This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request. |
575d2dd
to
a6232c7
Compare
a6232c7
to
a4513c2
Compare
This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request. |
Tested, working as expected. |
a4513c2
to
0692606
Compare
Signed-off-by: dushimsam <dushsam@gmail.com>
0692606
to
c36b317
Compare
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.
Changes looks good.
Description
Added the API to set a specific license as the main license for the given Upload
Changes
UploadController
to handle the logic.index.php
) by adding a new routePOST
/uploads/{id}/licenses/main
.openapi.yaml
file to introduce a new API.How to test
Make a POST request on the endpoint:
/uploads/{id}/licenses/main
.Screenshots
Related Issue:
Fixes #2459
cc: @shaheemazmalmmd @GMishx