-
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): Get removable folder contents #2551
Conversation
339e740
to
1084a7f
Compare
This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request. |
1084a7f
to
02f9f96
Compare
This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request. |
Signed-off-by: dushimsam <dushsam@gmail.com>
Signed-off-by: Gaurav Mishra <mishra.gaurav@siemens.com>
02f9f96
to
1fffdd4
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. Tested, working as expected.
Update API version after merge of: fossology#2551 fossology#2552 fossology#2553 fossology#2555 fossology#2562 fossology#2564 fossology#2565 fossology#2563 fossology#2568 fossology#2574 Signed-off-by: Gaurav Mishra <mishra.gaurav@siemens.com>
Update API version after merge of: fossology#2551 fossology#2552 fossology#2553 fossology#2555 fossology#2562 fossology#2564 fossology#2565 fossology#2563 fossology#2568 fossology#2574 Signed-off-by: Gaurav Mishra <mishra.gaurav@siemens.com>
Description
Added the API to retrieve the get a list of removable contents in a folder.
Changes
FolderController
to build the functionality.index.php
) by adding a new routeGET
/folders/{id}/contents/removable
.openapi.yaml
file to write the new API's documentation.How to test
Make a GET request on the endpoint:
/folders/{id}/contents/removable
.Screenshots
Related Issue:
Fixes #2540
cc: @shaheemazmalmmd @GMishx