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(bug) : Releases attachment download issue fix #2140

Merged
merged 1 commit into from
Oct 11, 2023

Conversation

keerthi-bl
Copy link
Contributor

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?

Issue: #2136

Suggest Reviewer

You can suggest reviewers here with an @mention.

How To Test?

How should these changes be tested by the reviewer?
Using postman.

URL : http://localhost:8080/resource/api/releases/caf6ff2180064d9faaebe8d7a7cd5ae2/attachments

Sample Response in the following attachment :

image

Have you implemented any additional tests?
No

Checklist

Must:

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

@keerthi-bl keerthi-bl changed the title feat(bug) : Download release attachment fail issue fix feat(bug) : Downloading release attachment fail issue fix Oct 6, 2023
@keerthi-bl keerthi-bl changed the title feat(bug) : Downloading release attachment fail issue fix feat(bug) : Release attachment downloading issue fix Oct 6, 2023
@keerthi-bl keerthi-bl changed the title feat(bug) : Release attachment downloading issue fix feat(bug) : Releases attachment download issue fix Oct 9, 2023
@nikkuma7
Copy link
Contributor

nikkuma7 commented Oct 9, 2023

Test successful.

image

@rudra-superrr rudra-superrr self-assigned this Oct 9, 2023
subsectionWithPath("_embedded.sw360:attachments.[]createdTeam").description("The attachment created team value"),
subsectionWithPath("_embedded.sw360:attachments.[]createdOn").description("The attachment created on value"),
subsectionWithPath("_embedded.sw360:attachments.[]checkStatus").description("The attachment checkStatus value"),
subsectionWithPath("_embedded.sw360:attachments.[]createdOn").description("The attachment checkedComment value"),
Copy link
Contributor

Choose a reason for hiding this comment

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

it should be []checkedComment according to the description.

@keerthi-bl keerthi-bl force-pushed the fix/AttachmentDownloadIssueFix branch from 6f49d28 to ec750b8 Compare October 10, 2023 05:37
@rudra-superrr
Copy link
Contributor

Code looks good.

@rudra-superrr rudra-superrr added ready ready to merge and removed needs code review labels Oct 11, 2023
@ag4ums ag4ums merged commit 1f9d918 into eclipse-sw360:main Oct 11, 2023
1 check 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.

4 participants