Skip to content

Hosted Adjunct - API Management#1102

Merged
p-kaczynski merged 5 commits intodevelopfrom
feature/1076/hosted_adjunct_api_mgmt
Feb 9, 2026
Merged

Hosted Adjunct - API Management#1102
p-kaczynski merged 5 commits intodevelopfrom
feature/1076/hosted_adjunct_api_mgmt

Conversation

@p-kaczynski
Copy link
Copy Markdown
Contributor

Impl #1076


POST/PUT Adjunct can now - instead of ExternalId - provide Origin property. If that happens, we set new property Ingesting to true and Error to (empty string), and a request is put to a new SQS queue.


Warning

Requires new queue (-adjuncts) and AWS__SQS__AdjunctQueueName setting for API to work correctly - otherwise ThrowIfNull triggers for Origin-having adjuncts.


  • Update Adjunct models with ingesting, error and origin
  • Ingest adjuncts with origin
  • Add migration for hosted adjuncts
  • Integration test for hosted adjunct
  • Apply PublicId to Hydra Adjunct

* origin only in hydra model
* also extract props common to Adjunct and Asset (ingesting ,error) to interface and update "clearing" extension method
also minor existing integration test fix
Includes some fixes to bugs found during testing
@p-kaczynski p-kaczynski requested a review from a team as a code owner February 4, 2026 16:01
@p-kaczynski p-kaczynski linked an issue Feb 4, 2026 that may be closed by this pull request
Comment thread src/protagonist/DLCS.Model/Messaging/IngestAdjunctRequest.cs
@griffri
Copy link
Copy Markdown
Contributor

griffri commented Feb 5, 2026

LGTM - although I'm not as close to the API's internals as the rest of the team, so it would be good to get their eyes on it.

Comment thread src/protagonist/DLCS.AWS/SQS/IQueueLookup.cs
Comment thread src/protagonist/DLCS.Model/Assets/AdjunctX.cs
Comment thread src/protagonist/DLCS.Model/Assets/DeliverableX.cs
@p-kaczynski p-kaczynski merged commit e7cbb9d into develop Feb 9, 2026
8 checks passed
@p-kaczynski p-kaczynski deleted the feature/1076/hosted_adjunct_api_mgmt branch February 9, 2026 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

API management of hosted adjuncts

4 participants