Skip to content

PO-1897 remove defendant account party#1639

Open
ShanCGI wants to merge 10 commits intomasterfrom
PO-1897-defendant-account-party-remove
Open

PO-1897 remove defendant account party#1639
ShanCGI wants to merge 10 commits intomasterfrom
PO-1897-defendant-account-party-remove

Conversation

@ShanCGI
Copy link
Copy Markdown
Contributor

@ShanCGI ShanCGI commented Mar 31, 2026

JIRA link (if applicable)

https://tools.hmcts.net/jira/browse/PO-1897

Change description

New API endpoint to delete a Defendant Account Party in Opal Fines DB

Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[x] No

@ShanCGI ShanCGI force-pushed the PO-1897-defendant-account-party-remove branch 4 times, most recently from e2ec6b4 to 2033cff Compare April 1, 2026 12:51
@ShanCGI ShanCGI force-pushed the PO-1897-defendant-account-party-remove branch 3 times, most recently from 53b6c67 to 400642f Compare April 7, 2026 09:53
@ShanCGI ShanCGI force-pushed the PO-1897-defendant-account-party-remove branch from 400642f to 3b0bcec Compare April 7, 2026 11:08
@ShanCGI ShanCGI marked this pull request as ready for review April 7, 2026 12:21
@ShanCGI ShanCGI requested review from a team April 7, 2026 12:21
@ShanCGI ShanCGI force-pushed the PO-1897-defendant-account-party-remove branch from 3b0bcec to b8ec421 Compare April 7, 2026 12:25
@ShanCGI ShanCGI force-pushed the PO-1897-defendant-account-party-remove branch from b8ec421 to d468dbb Compare April 15, 2026 14:36
@ShanCGI ShanCGI force-pushed the PO-1897-defendant-account-party-remove branch from d468dbb to 915ed5c Compare April 15, 2026 14:47
@ShanCGI ShanCGI force-pushed the PO-1897-defendant-account-party-remove branch from 915ed5c to 1b815d2 Compare April 17, 2026 10:47
@ShanCGI ShanCGI force-pushed the PO-1897-defendant-account-party-remove branch 2 times, most recently from fab8133 to 72d14cf Compare April 20, 2026 22:42
Comment thread src/test/java/uk/gov/hmcts/opal/service/DefendantAccountPartyServiceTest.java Outdated
ShanCGI and others added 9 commits April 21, 2026 14:07
Remove Defendant Account Party
Added integration test
- added Bearer Token authentication to the collection so this can be inherited for requests instead of having to define for each.
- Service Layer Pattern - Defendant Account service layer is the only place talking to repositories.
- use JPAs mechanism to auto increment the version number of the parent Defendant Account instead of using bumpVersion(...).
@ShanCGI ShanCGI force-pushed the PO-1897-defendant-account-party-remove branch from 6c90336 to 5116adb Compare April 21, 2026 13:15
@ShanCGI ShanCGI force-pushed the PO-1897-defendant-account-party-remove branch from 5116adb to 64963a7 Compare April 21, 2026 13:34
Copy link
Copy Markdown
Contributor

@Sean-Reed-CGI Sean-Reed-CGI left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants