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(CR): Disable Clearing Request creation for the projects which have linked releases without SRC type attachment #2263

Conversation

akshitjoshii
Copy link
Contributor

@akshitjoshii akshitjoshii commented Jan 10, 2024

Clearing Request will not be created if a project has 1 or more linked releases that do not have SRC type attachment,

Issue: #2259
Closes: #2259

How To Test?

Create a CR for a project.

Checklist

Must:

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

@akshitjoshii akshitjoshii force-pushed the feat/disableCreatingCRForProjectsWithLinkedReleasesWithoutSrcTypeAttachment branch from 1e9b759 to 03b1984 Compare January 10, 2024 07:23
@ag4ums ag4ums added needs code review needs general test This is general testing, meaning that there is no org specific issue to check for labels Jan 10, 2024
@sameed20
Copy link
Contributor

Testing this PR.

@sameed20
Copy link
Contributor

Test Successful
pr_testing

@rudra-superrr rudra-superrr self-assigned this Jan 18, 2024
List<Release> releasesWithoutSRC = new ArrayList<>();
Map<String, ProjectReleaseRelationship> releaseIdToUsage = project.getReleaseIdToUsage();
Set<String> releaseIdsSet = releaseIdToUsage.keySet();
List<String> releaseIdsList = new ArrayList<>(releaseIdsSet);
Copy link
Contributor

Choose a reason for hiding this comment

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

Directly store the Map keys in list.
List releaseIdsList = new ArrayList<>(releaseIdToUsage.keySet());

@@ -198,6 +198,7 @@ clearing.expert=Clearing Expert
clearing.progress=Clearing Progress
clearing.report=Clearing Report
clearing.request=Clearing Request
one.or.more.linked.releases.do.not.have.src.type.attachment=One or more linked releases do not have SRC type attachment.
Copy link
Contributor

Choose a reason for hiding this comment

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

add this in all language.properties files

…ve linked releases without SRC type attachment

Signed-off-by: Akshit Joshi <akshit.joshi@siemens-healthineers.com>
@akshitjoshii akshitjoshii force-pushed the feat/disableCreatingCRForProjectsWithLinkedReleasesWithoutSrcTypeAttachment branch from 03b1984 to 1ab1435 Compare January 18, 2024 09:37
@akshitjoshii
Copy link
Contributor Author

Hi @rudra-superrr, I have made the changes as you suggested.

@rudra-superrr
Copy link
Contributor

code looks good.

@rudra-superrr rudra-superrr added ready ready to merge and removed needs code review needs general test This is general testing, meaning that there is no org specific issue to check for labels Jan 18, 2024
@arunazhakesan arunazhakesan merged commit d77beb5 into eclipse-sw360:main Jan 29, 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.

Disable Create CR option for a project if any of its linked release does not have an attachment of type SRC.
5 participants