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

fix(rest): Added packageIds in project create and update APIs. #2885

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

nikkuma7
Copy link
Contributor

@nikkuma7 nikkuma7 commented Jan 23, 2025

Please provide a summary of your changes here.

  • Which issue is this pull request belonging to and how is it solving it? (Refer to issue here)
  • Did you add or update any new dependencies that are required for your change?

Closes #2876

Suggest Reviewer

You can suggest reviewers here with an @mention.

How To Test?

(post method ): http://localhost:8080/resource/api/projects
(patch method): http://localhost:8080/resource/api/projects/{Id}

While creating a project, include the packageIds and verify whether they are updated in the database. Similarly, while patching a project, ensure that the packageIds are saved in the database.

Checklist

Must:

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

@GMishx GMishx added needs code review needs general test This is general testing, meaning that there is no org specific issue to check for labels Jan 23, 2025
GMishx
GMishx previously approved these changes Jan 23, 2025
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.

@nikkuma7
Copy link
Contributor Author

@GMishx , Build issue resolved.

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.

LGTM

@GMishx GMishx merged commit fbea70a into eclipse-sw360:main Jan 31, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs general test This is general testing, meaning that there is no org specific issue to check for
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Need packageIds key field in create project rest API
2 participants