Skip to content

feat(assets): add the ability to disassociate an uploaded asset from a well#739

Merged
TylerAdamMartinez merged 2 commits into
stagingfrom
feature/tam-bdms-922-disassociate-uploaded-asset
Jun 24, 2026
Merged

feat(assets): add the ability to disassociate an uploaded asset from a well#739
TylerAdamMartinez merged 2 commits into
stagingfrom
feature/tam-bdms-922-disassociate-uploaded-asset

Conversation

@TylerAdamMartinez

Copy link
Copy Markdown
Contributor

Why

This PR addresses the following problem/context:

  • Add an endpoint to update an Asset's Thing association without modifying the Asset itself.
  • Allow assets to be reassigned to a different Thing or disassociated entirely.

How

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

  • Add a new PATCH /asset/{asset_id}/association endpoint.
  • Add AssetAssociationUpdate and AssetAssociationResponse schemas.
  • Add audit logging when creating new associations

Notes

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

  • This endpoint only updates the Asset <-> Thing association; it does not modify Asset metadata.
  • Existing associations are removed before creating a new one, ensuring an Asset is associated with at most one Thing.
  • Passing thing_id: null removes all Thing associations while preserving the Asset record.

@TylerAdamMartinez TylerAdamMartinez self-assigned this Jun 23, 2026
@TylerAdamMartinez TylerAdamMartinez added the enhancement New feature or request label Jun 23, 2026
@TylerAdamMartinez TylerAdamMartinez changed the title BDMS-922: Need a way to disassociate an uploaded asset from a well feat(assets): BDMS-922 Need a way to disassociate an uploaded asset from a well Jun 23, 2026
@TylerAdamMartinez TylerAdamMartinez changed the title feat(assets): BDMS-922 Need a way to disassociate an uploaded asset from a well feat(assets): add the ability to disassociate an uploaded asset from a well Jun 23, 2026
@TylerAdamMartinez TylerAdamMartinez marked this pull request as ready for review June 23, 2026 18:19

@likithabommasani21 likithabommasani21 left a comment

Copy link
Copy Markdown
Contributor

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 d5caa14 into staging Jun 24, 2026
10 checks passed
@TylerAdamMartinez TylerAdamMartinez deleted the feature/tam-bdms-922-disassociate-uploaded-asset branch June 24, 2026 15:43
@TylerAdamMartinez TylerAdamMartinez restored the feature/tam-bdms-922-disassociate-uploaded-asset branch June 25, 2026 19:12
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

None yet

Development

Successfully merging this pull request may close these issues.

2 participants