Skip to content

fix(api/asset): patch delete asset notifications#742

Merged
TylerAdamMartinez merged 1 commit into
stagingfrom
fix/tam-bdms-922-add-notify_edit_event
Jun 25, 2026
Merged

fix(api/asset): patch delete asset notifications#742
TylerAdamMartinez merged 1 commit into
stagingfrom
fix/tam-bdms-922-add-notify_edit_event

Conversation

@TylerAdamMartinez

Copy link
Copy Markdown
Contributor

Why

This PR addresses the following problem/context:

  • Asset deletions were not passing the authenticated user to the deletion helper.

How

Implementation summary - the following was changed/added/removed:

  • Updated the asset delete endpoint to pass the authenticated user into model_deleter()

Notes

Any special considerations, workarounds, or follow-up work to note?

  • N/A

@TylerAdamMartinez TylerAdamMartinez self-assigned this Jun 25, 2026
@TylerAdamMartinez TylerAdamMartinez added the bug Something isn't working label Jun 25, 2026
@TylerAdamMartinez TylerAdamMartinez changed the title fix(api/asset): Add delete asset notifications fix(api/asset): add delete asset notifications Jun 25, 2026
@TylerAdamMartinez TylerAdamMartinez changed the title fix(api/asset): add delete asset notifications fix(api/asset): patch delete asset notifications Jun 25, 2026
@TylerAdamMartinez TylerAdamMartinez marked this pull request as ready for review June 25, 2026 19:22

@chasetmartin chasetmartin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me

@TylerAdamMartinez TylerAdamMartinez merged commit 886cfc1 into staging Jun 25, 2026
11 of 12 checks passed
@TylerAdamMartinez TylerAdamMartinez deleted the fix/tam-bdms-922-add-notify_edit_event branch June 25, 2026 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants