8000 feat(api): clearing status by soham4abc · Pull Request #2288 · fossology/fossology · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat(api): clearing status #2288

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Feb 27, 2023
Merged

feat(api): clearing status #2288

merged 2 commits into from
Feb 27, 2023

Conversation

soham4abc
Copy link
Contributor
@soham4abc soham4abc commented Aug 10, 2022 •
edited by gitpod-io bot
Loading

Description

Clearing status parameter for the /uploads/ID/licenses endpoint.

How to test

Send a licenses request for an upload to see the response.

Screenshot

image

Fixes #2283

PTAL @GMishx

GMishx
GMishx requested changes Aug 12, 2022
View reviewed changes
@soham4abc soham4abc requested a review from GMishx August 12, 2022 13:53
@soham4abc
Copy link
Contributor Author

PTAL @GMishx

shaheemazmalmmd
shaheemazmalmmd reviewed Aug 25, 2022
View reviewed changes
@soham4abc soham4abc requested review from shaheemazmalmmd and removed request for GMishx August 25, 2022 18:07
@soham4abc
Copy link
Contributor Author
soham4abc commented Aug 25, 2022 •
edited
Loading

@GMishx @shaheemazmalmmd this PR is updated with ItemTreeBounds integration PTAL!!

@soham4abc soham4abc requested review from GMishx and removed request for shaheemazmalmmd August 25, 2022 18:09
@soham4abc
Copy link
Contributor Author

can we mark this PR as Work in progress?? @GMishx @shaheemazmalmmd

@soham4abc soham4abc force-pushed the clearing_decision branch 4 times, most recently from acca4ff to 70fd429 Compare August 30, 2022 17:00
@soham4abc soham4abc requested review from shaheemazmalmmd and removed request for GMishx August 30, 2022 17:01
@soham4abc
Copy link
Contributor Author

PTAL @GMishx @shaheemazmalmmd

@soham4abc soham4abc mentioned this pull request Sep 14, 2022
Open
@github-actions github-actions bot added the has merge conflicts PR to be rebased label Sep 27, 2022
@github-actions
Copy link

This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request.

@GMishx GMishx added this to the 4.3.0 milestone Nov 9, 2022
soham4abc and others added 2 commits February 27, 2023 18:34
@soham4abc @GMishx
feat(api): clearing status
0fd6be4
@GMishx
fix(api): add new model FileLicenses for REST API
eb5d5e0 
Signed-off-by: Gaurav Mishra <mishra.gaurav@siemens.com>
@github-actions github-actions bot removed the has merge conflicts PR to be rebased label Feb 27, 2023
GMishx
GMishx approved these changes Feb 27, 2023
View reviewed changes
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.
Tested, working as expected.

@GMishx GMishx merged commit 267337d into fossology:master Feb 27, 2023
@soham4abc
Copy link
Contributor Author

@GMishx Thanks a lot!

@GMishx
Copy link
Member
GMishx commented Feb 27, 2023

@GMishx Thanks a lot!

Thanks for your contribution @soham4abc

@deveaud-m deveaud-m mentioned this pull request Aug 3, 2023
Open
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GMishx GMishx GMishx approved these changes

@shaheemazmalmmd shaheemazmalmmd Awaiting requested review from shaheemazmalmmd

Assignees
No one assigned
Labels
needs code review needs test
Projects
None yet
Milestone
4.3.0
Development

Successfully merging this pull request may close these issues.

Creaate REST API for the clearing status and cleared/ToBeCleared/Total response.
3 participants
@soham4abc @GMishx @shaheemazmalmmd
0