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 code to get obligation releaseView data in project. #2486

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nikkuma7
Copy link
Contributor

@nikkuma7 nikkuma7 commented Jun 14, 2024

Description

fix(rest): Added code to get obligation releaseView data in project.

http://localhost:8080/resource/api/projects/{ id}/licenseObligations?view=true

Closes : #2485

Please describe your issue in few words here.

How to reproduce

Describe the bug and list the steps you used when the issue occurred.

Screenshots ( if applicable )

Add screenshots to help explain your problem.

Versions

  • Docker version OR
  • Last commit id on main

SW360 logs

  • With docker through docker logs sw360
  • From bare metal / vm install system under /var/log/sw360/sw360.log and /var/log/tomcat/error.log

@nikkuma7 nikkuma7 force-pushed the fix/ObligationReleaseView branch from a2e50ff to 79c984e Compare June 14, 2024 04:55
@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

@nikkuma7 please rebase with latest main

@nikkuma7 nikkuma7 force-pushed the fix/ObligationReleaseView branch 2 times, most recently from 1f2fc57 to cd59cd7 Compare September 10, 2024 08:04
GMishx
GMishx previously approved these changes Oct 17, 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.

@GMishx
Copy link
Member

GMishx commented Oct 17, 2024

@deo002 please test this PR.

@deo002
Copy link

deo002 commented Oct 22, 2024

status value is null for obligations in response

@nikkuma7 nikkuma7 force-pushed the fix/ObligationReleaseView branch 2 times, most recently from b2861ba to 9549846 Compare October 22, 2024 08:50
@nikkuma7 nikkuma7 force-pushed the fix/ObligationReleaseView branch from 9549846 to 88d100a Compare October 29, 2024 09:35
@nikkuma7
Copy link
Contributor Author

status value is null for obligations in response

image

Addressed comment.

@GMishx GMishx added this to the Release - 20.0.0 milestone Nov 27, 2024
@GMishx
Copy link
Member

GMishx commented Dec 4, 2024

@deo002 please test this PR again.

@nikkuma7 please rebase.

@GMishx GMishx added the has merge conflicts The PR has merge conflicts label Dec 4, 2024
@nikkuma7 nikkuma7 force-pushed the fix/ObligationReleaseView branch from 88d100a to f0960ba Compare December 12, 2024 04:31
@nikkuma7 nikkuma7 removed the has merge conflicts The PR has merge conflicts label Dec 12, 2024
@GMishx GMishx added the has merge conflicts The PR has merge conflicts label Dec 23, 2024
Signed-off-by: Nikesh kumar <kumar.nikesh@siemens.com>
@nikkuma7 nikkuma7 force-pushed the fix/ObligationReleaseView branch from f0960ba to 7d7d69b Compare December 24, 2024 09:01
@nikkuma7 nikkuma7 removed the has merge conflicts The PR has merge conflicts label Dec 24, 2024
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.

fix(rest): Added code to get obligation releaseView data in project.
3 participants