-
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): Import obligation list from CSV #2563
Conversation
cbe53c4
to
ca1ea2e
Compare
This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request. |
ca1ea2e
to
c433eee
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>
c433eee
to
6979600
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 import obligation list from a CSV file.
Changes
LicenseController
to build the functionality.index.php
) by adding a new routePOST
/license/obligation/import-csv
.openapi.yaml
file to write the new API's documentation.How to test
Make a POST request on the endpoint:
/license/obligation/import-csv
,Screenshots
Related Issue:
Fixes #2560
cc: @shaheemazmalmmd @GMishx