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): New endpoint split components. #2007

Merged
merged 2 commits into from
Jul 19, 2023

Conversation

muhammadali9699
Copy link
Contributor

Please provide a summary of your changes here.
This PR is to split two components

Issue: #1975

Suggest Reviewer

You can suggest reviewers here with an @mention.

How To Test?

Create two components
Get components from couchdb and pass it to the api as a body
add body of component with changes made in component (sample given below)

{
"targetComponent": {
"id": "7cefb12b95c34ba1b6344b7cc4049179",
"rev": "1-cd3b83c14b51c200bb25b88e88c6931b",
"type": "component",
"name": "Deemo comp 4",
"description": "",
"attachments": [],
"createdOn": "2023-05-17",
"componentType": "CODE_SNIPPET",
"createdBy": "admin@sw360.org",
"subscribers": [],
"moderators": [],
"componentOwner": "",
"ownerAccountingUnit": "",
"ownerGroup": "",
"ownerCountry": "",
"roles": {},
"visbility": "EVERYONE",
"businessUnit": "DEPARTMENT",
"externalIds": {},
"additionalData": {},
"releaseIds": [],
"mainLicenseIds": [],
"defaultVendorId": "",
"categories": [
"virtual-machine"
],
"languages": [],
"softwarePlatforms": [],
"operatingSystems": [],
"vendorNames": [],
"homepage": "",
"mailinglist": "",
"wiki": "",
"blog": "",
"modifiedBy": "",
"modifiedOn": ""
},
"srcComponent": {
"id": "99ab81a8e8cc45f3bee1ba6f0f928a2a",
"rev": "1-cd3b83c14b51c200bb25b88e88c6931b",
"type": "component",
"name": "Deemo comp 4",
"description": "",
"attachments": [],
"createdOn": "2023-05-17",
"componentType": "CODE_SNIPPET",
"createdBy": "admin@sw360.org",
"subscribers": [],
"moderators": [],
"componentOwner": "",
"ownerAccountingUnit": "",
"ownerGroup": "",
"ownerCountry": "",
"roles": {},
"visbility": "EVERYONE",
"businessUnit": "DEPARTMENT",
"externalIds": {},
"additionalData": {},
"releaseIds": [],
"mainLicenseIds": [],
"defaultVendorId": "",
"categories": [
"virtual-machine"
],
"languages": [],
"softwarePlatforms": [],
"operatingSystems": [],
"vendorNames": [],
"homepage": "",
"mailinglist": "",
"wiki": "",
"blog": "",
"modifiedBy": "",
"modifiedOn": ""
}
}

Screenshot:

Screenshot 2023-06-19 150138

@ag4ums ag4ums added needs code review needs general test This is general testing, meaning that there is no org specific issue to check for REST New-UI Level for the API and UI level changes for the new-ui labels Jun 26, 2023
@nikkuma7
Copy link

Test successful.
image

@eldrin30
Copy link
Contributor

eldrin30 commented Jul 4, 2023

@muhammadali9699, kindly add documentation and resolve conflicts

@muhammadali9699 muhammadali9699 force-pushed the feat/api/splitComponents branch 4 times, most recently from 7e23555 to bedfc22 Compare July 7, 2023 02:34
@muhammadali9699 muhammadali9699 force-pushed the feat/api/splitComponents branch 2 times, most recently from 9a9c181 to 7a88723 Compare July 7, 2023 09:27
Signed-off-by: Muhammad Ali <alimuhammad@siemens.com>
@eldrin30
Copy link
Contributor

Code looks good, just rebase the commit

@ag4ums ag4ums 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 Jul 19, 2023
@ag4ums ag4ums merged commit 50e4f07 into eclipse-sw360:main Jul 19, 2023
1 check passed
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.

5 participants