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): includeAllAttachments parameter in licenseInfo endpoint #2488

Merged

Conversation

afsahsyeda
Copy link
Contributor

Description

The parameter "includeAllAttachments" when set to true generates license info file for a project including both CLX and ISR attachment of a release (only one attachment per release is considered and CLX is preferred).

How to reproduce

Query the GET endpoint for licenseInfo of a project.

Screenshots ( if applicable )

image

@afsahsyeda afsahsyeda force-pushed the feat/GenerateReadmeossFromISR branch 2 times, most recently from 70416d9 to 0c3f6e0 Compare June 19, 2024 07:45
@GMishx GMishx added needs code review has merge conflicts needs general test This is general testing, meaning that there is no org specific issue to check for labels Aug 16, 2024
@GMishx
Copy link
Member

GMishx commented Aug 16, 2024

@afsahsyeda please rebase with latest main to get the REST test cases.

Copy link
Member

@GMishx GMishx left a comment

Choose a reason for hiding this comment

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

Changes looks good.

Signed-off-by: afsahsyeda <afsah.syeda@siemens-healthineers.com>
@afsahsyeda
Copy link
Contributor Author

This feature is live in our production environment and is working fine.

@afsahsyeda afsahsyeda added ready ready to merge and removed needs general test This is general testing, meaning that there is no org specific issue to check for labels Sep 11, 2024
@GMishx GMishx merged commit dfefe8a into eclipse-sw360:main Sep 12, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants