-
Notifications
You must be signed in to change notification settings - Fork 399
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(api): Get list of bulk-history API #2481
Conversation
This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor changes.
Tested, working as expected. |
d804cd2
to
34d2349
Compare
34d2349
to
11d755a
Compare
This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request. |
1 similar comment
This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes looks good.
@dushimsam , please rebase for merge. |
11d755a
to
a379e1b
Compare
Done |
a379e1b
to
a304c3e
Compare
This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request. |
Signed-off-by: dushimsam <dushsam@gmail.com>
a304c3e
to
ea3adb3
Compare
Description
Added the API to a get a list of the bulk history for a particular item.
Changes
UploadTreeController
to handle the logic.index.php
) by adding a new routeGET
/uploads/{id}/items/{itemId}/bulk-history
.openapi.yaml
file to write the new API's documentation.How to test
Make a GET request on the endpoint:
GET
/uploads/{id}/items/{itemId}/bulk-history
.,Screenshots
Related Issue:
Fixes #2472
cc: @shaheemazmalmmd @GMishx