Skip to content

Immediate completion of none channel#1170

Merged
JackLewis-digirati merged 8 commits intodevelopfrom
feature/immediateNoneChannelReturn
Apr 14, 2026
Merged

Immediate completion of none channel#1170
JackLewis-digirati merged 8 commits intodevelopfrom
feature/immediateNoneChannelReturn

Conversation

@JackLewis-digirati
Copy link
Copy Markdown
Collaborator

@JackLewis-digirati JackLewis-digirati commented Apr 8, 2026

Resolves #1154

What does this change?

This PR modifies the None delivery channel to complete without needing to be ingested - this PR will also raise a completed batch notification immediately if required

@JackLewis-digirati JackLewis-digirati force-pushed the feature/immediateNoneChannelReturn branch from 3d4b769 to f5b228c Compare April 10, 2026 10:12
@JackLewis-digirati JackLewis-digirati marked this pull request as ready for review April 10, 2026 16:39
@JackLewis-digirati JackLewis-digirati requested a review from a team as a code owner April 10, 2026 16:39
Comment thread src/protagonist/DLCS.Model/Assets/ImageDeliveryChannelX.cs Outdated
Comment thread src/protagonist/Engine/Ingest/IngestExecutor.cs
Comment thread src/protagonist/API/Features/Image/Ingest/DeliveryChannelProcessor.cs Outdated
Comment thread src/protagonist/Engine/Ingest/IngestExecutor.cs
Comment thread src/protagonist/API/Features/Image/Ingest/AssetProcessor.cs Outdated
@JackLewis-digirati JackLewis-digirati changed the title Initial immediate completion of none channel immediate completion of none channel Apr 13, 2026
@JackLewis-digirati JackLewis-digirati changed the title immediate completion of none channel Immediate completion of none channel Apr 13, 2026
- Move updating delivery channel details into  CreateOrUpdateImage
- Create a repository class to work with ImageStorage
- Make sure image storage record updated
- Additional tests to cover
Comment thread src/protagonist/Engine/Data/EngineAssetRepository.cs Outdated
Comment thread src/protagonist/DLCS.Repository.Tests/Storage/ImageStorageXTests.cs
Comment thread src/protagonist/API/Features/Image/Requests/CreateOrUpdateImage.cs
Comment thread src/protagonist/API/Features/Image/Ingest/AssetProcessor.cs Outdated
Comment thread src/protagonist/API/Features/Image/Ingest/AssetProcessor.cs Outdated
- Add additional comments
- Remove commented code
@JackLewis-digirati JackLewis-digirati merged commit 7a3f5cc into develop Apr 14, 2026
8 checks passed
@JackLewis-digirati JackLewis-digirati deleted the feature/immediateNoneChannelReturn branch April 14, 2026 11:24
donaldgray added a commit that referenced this pull request Apr 14, 2026
Rebased as PR #1170 had been merged
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 auto-finish none channel

2 participants