-
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): Export Obligation list as CSV #2574
Conversation
This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request. |
edadb70
to
5630d6b
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>
5630d6b
to
8d44d98
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 export Obligations' list as a CSV file.
Changes
LicenseController
to build the functionality.index.php
) by adding a new routeGET
/license/obligation/export-csv
.openapi.yaml
file to write the new API's documentation.How to test
Make a GET request on the endpoint:
/license/obligation/export-csv
,Screenshots
Related Issue:
Fixes #2560
cc: @shaheemazmalmmd @GMishx