Framework D changes 26.20 release - #488
Open
vmuthuhm wants to merge 21 commits into
Open
Conversation
* PEG-2848 Testing Framework-D improvement * PEG-2848 Testing Framework-D improvement * Update pom.xml
* PEG-2848 Testing Framework-D improvement * PEG-2848 Testing Framework-D improvement * Update pom.xml
…plicantDetails signature
…ce with ReferenceDataService mock
…Framework D changes - Restore processReplay, sendPostalNotification(5-arg), courtDocumentMetadata, startDate in getFutureHearings to release-engg-2609 versions of DefendantsAddedToCourtProceedingsProcessor, PostalService, NotificationService, ListingService (PEG-2848 had removed these; they must be kept from base) - Apply Framework D (JsonObjects.createObjectBuilder) on restored sources - Fix ListingStub: update setupListingAnyAllocationQuery/setupListingAnyFutureAllocationQuery to 3/4-arg signatures matching release-engg-2609 - Fix HearingNotificationHelperTest: restore full release-engg-2609 version + Framework D changes - Fix InitiateCourtApplicationProceedingsCommandApiTest: add linkType STANDALONE to buildEnvelope Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ebase onto team/PEG-2848-R20 Convert remaining javax.json.Json usage to uk.gov.justice.services.messaging.JsonObjects across the codebase, restore imports dropped during rebase auto-merge (Hearing, HearingInitiateEnriched, CourtApplication, PrisonCourtRegisterGeneratedV2, MetadataBuilder, Instant, Response, etc.), and add the missing progression-domain-message (raml classifier) dependency to progression-command-handler's pojo-generation-plugin config so UpdateDefendantListingStatusV3 generates correctly - a pre-existing gap that silently skipped generation for any schema referencing a $ref owned by another module.
vmuthuhm
requested review from
UdayaPriyaSunkara1,
mahmcts and
noelkeoghan2
and removed request for
a team
July 21, 2026 19:08
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Change description
Testing done
Security Vulnerability Assessment
CVE Suppression: Are there any CVEs present in the codebase (either newly introduced or pre-existing) that are being intentionally suppressed or ignored by this commit?
Checklist