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): saveUsages in project page #2372

Merged
merged 1 commit into from
Oct 21, 2024

Conversation

rudra-superrr
Copy link
Contributor

@rudra-superrr rudra-superrr commented Mar 21, 2024

Issue: Closes #2371

Description: this new endpoint is used to save attachmnetUsages for the project.

How to test:
Pass Map< String, List< String >> in request body.
String can be of 4 types:

  • selected : release attachmentUsage that you want to save/check
  • deselected : release attachmentUsage that you want to delete/uncheck
  • selectedConcludedUsages : release attachmentUsage that you want to save/check of type license info's conclusion checkbox only
  • deselectedConcludedUsages: release attachmentUsage that you want to delete/uncheck of type license info's conclusion checkbox only

Format of String inside list varies according to the usageFields (sourcePackage, licenseInfo, manuallySet) :

  • for sourcePackage, releaseId_sourcePakage_attachmentContentId
  • for licenseInfo, projectId-releaseId_licenseInfo_attachmentContentId (for directly linked project) OR projectPath-releaseId_licenseInfo_attachmentContentId (for nested projects)
  • for manuallySet, releaseId_manuallySet_attachmentContentId

Example:
{"selected": ["4427a8e723ad405db63f75170ef240a2_sourcePackage_5c5d6f54ac6a4b33bcd3c5d3a8fefc43", "value2"],
"deselected": ["de213309ba0842ac8a7251bf27ea8f36_manuallySet_eec66c3465f64f0292dfc2564215c681", "value2"],
"selectedConcludedUsages": ["de213309ba0842ac8a7251bf27ea8f36_licenseInfo_eec66c3465f64f0292dfc2564215c681"],"
"deselectedConcludedUsages":
["ade213309ba0842ac8a7251bf27ea8f36_licenseInfo_aeec66c3465f64f0292dfc2564215c681"]}

@rudra-superrr rudra-superrr added needs code review needs general test This is general testing, meaning that there is no org specific issue to check for REST labels Mar 21, 2024
@heliocastro heliocastro added the New-UI Level for the API and UI level changes for the new-ui label Mar 27, 2024
@rudra-superrr rudra-superrr force-pushed the feat/saveUsages branch 2 times, most recently from 648c302 to 822dc33 Compare April 23, 2024 08:19
@keerthi-bl
Copy link
Contributor

Testing this PR.

@keerthi-bl
Copy link
Contributor

Test successfull.

PR_2371_SUCCESS_VALIDATION

PR_2371_SUCCESS_FAILURE

GMishx
GMishx previously approved these changes Aug 16, 2024
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. Please rebase with latest main.

@rudra-superrr rudra-superrr force-pushed the feat/saveUsages branch 2 times, most recently from b93a070 to 86950a6 Compare August 19, 2024 06:24
@GMishx
Copy link
Member

GMishx commented Aug 31, 2024

@keerthi-bl please retest the branch after rebase.

@rudra-superrr
Copy link
Contributor Author

Rebase completed, now you can retest this @keerthi-bl .

Signed-off-by: Rudra Chopra <prabhuchopra@gmail.com>
@keerthi-bl
Copy link
Contributor

Test successfull.

image

@GMishx GMishx 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 Oct 21, 2024
@GMishx GMishx merged commit 062a892 into eclipse-sw360:main Oct 21, 2024
2 checks passed
@GMishx GMishx deleted the feat/saveUsages branch October 21, 2024 10:37
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.

Save attachment usages endpoint for project
6 participants