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

Add an endpoint for deleting images by entity ID #104

Open
2 tasks
VKTB opened this issue Jan 29, 2025 · 0 comments · May be fixed by #117
Open
2 tasks

Add an endpoint for deleting images by entity ID #104

VKTB opened this issue Jan 29, 2025 · 0 comments · May be fixed by #117
Assignees
Labels
enhancement New feature or request

Comments

@VKTB
Copy link
Collaborator

VKTB commented Jan 29, 2025

Dependent on #38. This endpoint should be a DELETE with a entity_id query parameter at /images that:

  • Deletes all images with that entity_id
  • Returns 204 if the entity_id is invalid or is not associated with any images

Will need consideration on how to deal with errors mid way, I expect they can be mitigated by deleting the database documents after successful deletion from the object store like we already do for the others.

@VKTB VKTB added the enhancement New feature or request label Jan 29, 2025
@VKTB VKTB self-assigned this Jan 29, 2025
@VKTB VKTB moved this to To Do On This Sprint in Inventory Management System Jan 29, 2025
@VKTB VKTB moved this from To Do On This Sprint to In Progress in Inventory Management System Jan 29, 2025
VKTB added a commit that referenced this issue Feb 11, 2025
VKTB added a commit that referenced this issue Feb 11, 2025
VKTB added a commit that referenced this issue Feb 11, 2025
@VKTB VKTB linked a pull request Feb 11, 2025 that will close this issue
5 tasks
@VKTB VKTB moved this from In Progress to In Review in Inventory Management System Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: In Review
Development

Successfully merging a pull request may close this issue.

1 participant