-
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): Remove a particular main license from an upload #2463
Conversation
7b648cc
to
7c1ca0f
Compare
@@ -837,4 +837,38 @@ public function getGroupsWithPermissions($request, $response, $args) | |||
$res["permGroups"] = $finalPermGroups; | |||
return $response->withJson($res, 200); | |||
} | |||
|
|||
//remove main license | |||
/*** |
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.
/*** | |
/** |
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.
Quick reminder.
17eeaf7
to
6c28191
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.
Another minor change needed.
This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request. |
6c28191
to
418da08
Compare
418da08
to
ab4bace
Compare
ab4bace
to
e8dbbba
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.
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.
Tested, working as expected.
e8dbbba
to
b8a3590
Compare
Description
Added the API to delete a specific main license from an upload.
Changes
UploadController
to handle the logic.index.php
) by adding a new routeDELETE
/uploads/{id}/licenses/{licenseId}/main
.openapi.yaml
file to introduce a new API.How to test
Make a DELETE request on the endpoint:
/uploads/{id}/licenses/{licenseId}/main
.Screenshots
Related Issue:
Fixes #2459
cc: @shaheemazmalmmd @GMishx