Skip to content
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(rest) - Rest API for downloading components report. #1965

Merged
merged 1 commit into from
Jul 17, 2023

Conversation

keerthi-bl
Copy link
Contributor

Please provide a summary of your changes here.

  • Did you add or update any new dependencies that are required for your change?

No

Issue:

Suggest Reviewer

You can suggest reviewers here with an @mention.

How To Test?

It can be tested using Rest client (Postman) with following URL and sample response.

URL : http://localhost:8080/resource/api/reports?withlinkedreleases=true&mimetype=xlsx&mailrequest=true&module=components

HTTPRequest type : GET

Header (Add Authorisation : Token avbxxxxxxxxxxxh)

Sample response :

{
"response": "E-mail sent succesfully to the end user.",
"url": "http://127.0.0.1:8080/resource/api/reports/[email protected]&module=components&extendedByReleases=true&token=/tmp/[email protected]/2023-05-17_a990c489-acf8-4600-911c-2d89015b8bba"
}

Checklist

Must:

  • All related issues are referenced in commit messages and in PR

@keerthi-bl
Copy link
Contributor Author

This PR can only be merged only after the PR - #1902 get merged.

@smrutis1
Copy link
Contributor

Please update the commit message

image

@ag4ums ag4ums added the WIP work in progress label May 17, 2023
@keerthi-bl keerthi-bl force-pushed the fix/componentsreport branch 4 times, most recently from c65ec23 to 007fa33 Compare June 13, 2023 07:28
@ag4ums ag4ums added needs code review needs general test This is general testing, meaning that there is no org specific issue to check for and removed WIP work in progress labels Jun 21, 2023
@keerthi-bl keerthi-bl force-pushed the fix/componentsreport branch 2 times, most recently from bd7b219 to 524df35 Compare June 27, 2023 07:46
@ag4ums ag4ums added REST New-UI Level for the API and UI level changes for the new-ui labels Jun 27, 2023
@nikkuma7
Copy link

nikkuma7 commented Jul 3, 2023

Test Successful.

image
image
image
image

@ag4ums ag4ums removed the needs general test This is general testing, meaning that there is no org specific issue to check for label Jul 10, 2023
Copy link
Contributor

@smrutis1 smrutis1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Requesting minor changes

Signed-off-by: Keerthi B L <[email protected]>

feat(rest): new end point for downloading project

Signed-off-by: Keerthi B L <[email protected]>

feat(rest):update project

feat(rest): end point for downloding my components report

Signed-off-by: Keerthi B L <[email protected]>
@keerthi-bl
Copy link
Contributor Author

Review comments addressed.

Copy link
Contributor

@smrutis1 smrutis1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CLGTM

@ag4ums ag4ums added ready ready to merge and removed needs code review labels Jul 17, 2023
@ag4ums ag4ums merged commit d8ecfcf into eclipse-sw360:main Jul 17, 2023
1 check passed
@GMishx GMishx deleted the fix/componentsreport branch July 21, 2023 05:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New-UI Level for the API and UI level changes for the new-ui ready ready to merge REST
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants