-
-
Notifications
You must be signed in to change notification settings - Fork 17
Update Album Delete Endpoint to Remove All Related ImagesΒ #47
Copy link
Copy link
Labels
APIauthbackendduplicateThis issue or pull request already existsThis issue or pull request already existsenhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomersjestonlydust-waveContribute to awesome OSS repos during OnlyDust's open source weekContribute to awesome OSS repos during OnlyDust's open source weektesting
Metadata
Metadata
Assignees
Labels
APIauthbackendduplicateThis issue or pull request already existsThis issue or pull request already existsenhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomersjestonlydust-waveContribute to awesome OSS repos during OnlyDust's open source weekContribute to awesome OSS repos during OnlyDust's open source weektesting
Type
Fields
Give feedbackNo fields configured for Task.
π Description
We need to update the album delete API endpoint so that when a user deletes an album:
π Current Problem
β What Needs to Be Done
π§ Logic (Simple Steps):
π§ͺ How to Test
Create an album and upload multiple images to it.
Then call the delete album endpoint.
β Check if the album is removed from the DB.
β Check if related images are deleted from both the DB and Cloudinary.
π‘ Additional Suggestions
Add error handling for:
Cloudinary failures (e.g., retry or log)
Missing images or album
Optional: Add a confirmation step or warning before permanent deletion (for UI).
Optional: Add logging for deleted files (filename, public ID, etc.)
β° Priority: High
This is important to prevent junk data and unnecessary Cloudinary usage, which can increase storage cost.
#Β£ π Related Endpoints
DELETE /api/v1/admin/album/delete/AlbumId=
DELETE /api/images (or wherever images are stored)