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

Delete related attachment & images when an entity is deleted #399

Open
joelvdavies opened this issue Oct 22, 2024 · 0 comments · May be fixed by #460
Open

Delete related attachment & images when an entity is deleted #399

joelvdavies opened this issue Oct 22, 2024 · 0 comments · May be fixed by #460
Assignees
Labels
enhancement New feature or request

Comments

@joelvdavies
Copy link
Collaborator

joelvdavies commented Oct 22, 2024

When a catalogue item, item or system is deleted. Any attachments/images should also be deleted. This relies on

I think the simplest method for authentication is to ensure that on the delete request ims-api does the following:

  • Send a request to delete all related entity data using the same jwt token and wait for the response and then
    • If successful: Delete the entity
    • If authentication fails producing a 403: Replicate the same error so the front end does the token refresh
    • If any other error occurs or a response times out: Raise an appropriate error - Will likely need additional logic to prevent existing tests from failing due to not being able to contact the object-storage-api.

Further considerations

  • May require integration tests
@joelvdavies joelvdavies added the enhancement New feature or request label Oct 22, 2024
@joelvdavies joelvdavies changed the title Delete related attachment & image files when an entity is deleted Delete related attachment & images when an entity is deleted Oct 22, 2024
@joelvdavies joelvdavies self-assigned this Jan 13, 2025
@joelvdavies joelvdavies moved this from Backlog to In Progress in Inventory Management System Jan 13, 2025
@joelvdavies joelvdavies moved this from In Progress to Up Next in Inventory Management System Jan 13, 2025
@joelvdavies joelvdavies assigned VKTB and unassigned joelvdavies Jan 13, 2025
@VKTB VKTB moved this from Up Next 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 Feb 12, 2025
VKTB added a commit that referenced this issue Feb 13, 2025
@VKTB VKTB linked a pull request Feb 13, 2025 that will close this issue
5 tasks
VKTB added a commit that referenced this issue Feb 13, 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 Progress
2 participants