Skip to content

Hdpi 3549 non rent backend api#1466

Open
sdmiddletonSolirius wants to merge 13 commits intomasterfrom
Hdpi-3549-non-rent-backend-api
Open

Hdpi 3549 non rent backend api#1466
sdmiddletonSolirius wants to merge 13 commits intomasterfrom
Hdpi-3549-non-rent-backend-api

Conversation

@sdmiddletonSolirius
Copy link
Copy Markdown
Contributor

Jira link

See Hdpi-3549

Change description

  • Branched off hdpi-3495-rent-arrears-screen-backend
  • Added disputeClaim and disputeDetails to DefendantResponses,
    migration V063, DefendantResponseEntity, and DefendantResponseService so they are stored in the draft and saved on final submit.

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?

  • Yes
  • No

Checklist

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • Does this PR introduce a breaking change

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 4, 2026

CCD diff summary

👉 Full report: https://github.com/hmcts/pcs-api/actions/runs/23750426498?check_suite_focus=true

ComplexTypes/2_DefendantResponses.json

ListElementCodeIDFieldTypeParameterFieldTypeElementLabel
+disputeClaim
+DefendantResponses
+
+YesOrNo
+ 
+disputeClaimDetails
+DefendantResponses
+
+Text
+ 

@tvr-hmcts tvr-hmcts self-requested a review March 18, 2026 09:15
Copy link
Copy Markdown
Contributor

@tvr-hmcts tvr-hmcts left a comment

Choose a reason for hiding this comment

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

Minor comment. Lets not forget the tests when applying these changes.

Copy link
Copy Markdown
Contributor

@tvr-hmcts tvr-hmcts left a comment

Choose a reason for hiding this comment

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

Couple of comments for alterations to the new code pushed.

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.

7 participants