Skip to content

HDPI-8448: Create WA task when defendant LR uploads additional docs - #2458

Open
scottstewart-sl wants to merge 13 commits into
masterfrom
HDPI-8448_lr_additional_docs_wa_task
Open

HDPI-8448: Create WA task when defendant LR uploads additional docs#2458
scottstewart-sl wants to merge 13 commits into
masterfrom
HDPI-8448_lr_additional_docs_wa_task

Conversation

@scottstewart-sl

@scottstewart-sl scottstewart-sl commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Jira link

See HDPI-8448

Change description

Create WA task when defendant LR uploads additional docs. This does not cover a distinction between the main claim and counter claims - that will be in a later ticket.

Also cleaning up some code and using an existing helper method from ListValueUtils instead of a custom method to extract values from a CCD list.

Screenshot showing WA tasks being created now:

image

Testing done

Manual testing on preview

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

PCS checklist

  • New functional test classes carry the gating annotations (@EnabledIfEnvironmentVariable(named = "CCD_ENABLED", ...) + shutter guard) unless they must run on unlabelled PRs (HDPI-8084)
  • Infrastructure / chart / suppression changes have a second pair of eyes from the service admins (HDPI-8153)

@github-actions

github-actions Bot commented Aug 20, 2026

Copy link
Copy Markdown

CCD diff summary

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

No change

@hmcts-jenkins-j-to-z
hmcts-jenkins-j-to-z Bot requested a deployment to preview August 20, 2026 14:02 Abandoned
@hmcts-jenkins-j-to-z
hmcts-jenkins-j-to-z Bot requested a deployment to preview August 20, 2026 15:05 Abandoned
@hmcts-jenkins-j-to-z
hmcts-jenkins-j-to-z Bot requested a deployment to preview August 20, 2026 15:46 Abandoned
);
camundaService.createTask(caseReference, TaskType.REVIEW_ADDITIONAL_DOCS_CLAIM, description);
}
createAdditionalDocsWATask(party, selectedGenApp, caseReference, mainClaim, documentEntities);

@scottstewart-sl scottstewart-sl Aug 20, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The removed code has been extracted as-is into a method below, which can then be used by the legal rep additional documents method

SERVICES_CCD_DATA_STORE_API: http://${SERVICE_NAME}-ccd-data-store-api
SERVICES_CCD_CASE_ASSIGNMENT_API: http://${SERVICE_NAME}-aac-manage-case-assignment
SERVICES_ROLE_ASSIGNMENT_API: http://${SERVICE_NAME}-am-role-assignment-service
# SERVICES_ROLE_ASSIGNMENT_API: http://${SERVICE_NAME}-am-role-assignment-service

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Temporary change

@scottstewart-sl
scottstewart-sl marked this pull request as ready for review August 21, 2026 09:22
@scottstewart-sl
scottstewart-sl requested review from a team, benouaer and guygrewal77 August 21, 2026 09:22
@scottstewart-sl
scottstewart-sl requested a review from a team as a code owner August 21, 2026 09:22
…onal_docs_wa_task

# Conflicts:
#	src/main/java/uk/gov/hmcts/reform/pcs/ccd/event/legalrepdocumentupload/LegalRepDocumentUpload.java
#	src/test/java/uk/gov/hmcts/reform/pcs/ccd/event/legalrepdocumentupload/LegalRepDocumentUploadTest.java
…onal_docs_wa_task

# Conflicts:
#	src/test/java/uk/gov/hmcts/reform/pcs/ccd/service/document/DocumentServiceTest.java
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.

4 participants