Skip to content

HDPI-6510: Gen Apps supporting docs#1853

Open
scottstewart-sl wants to merge 2 commits into
HDPI-4573_view_applicationsfrom
HDPI-6510_applications_supporting_docs
Open

HDPI-6510: Gen Apps supporting docs#1853
scottstewart-sl wants to merge 2 commits into
HDPI-4573_view_applicationsfrom
HDPI-6510_applications_supporting_docs

Conversation

@scottstewart-sl
Copy link
Copy Markdown
Contributor

@scottstewart-sl scottstewart-sl commented May 19, 2026

Jira link

See HDPI-6510

Change description

Rename supporting documents that were uploaded as part of a citizen gen app, to include the gen app number and defendant label as a postfix before the file extension.

Related pcs-frontend PR: hmcts/pcs-frontend#1381

Testing done

Local testing with corresponding pcs-frontend changes

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

Signed-off-by: sstewart <202802468+scottstewart-sl@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 19, 2026

CCD diff summary

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

CaseField.json

RegularExpressionMaxSearchableMinFieldTypeParameterHintTextLabelIDFieldType
+
+
+
+
+GeneralApplication
+
+ 
+genApps
+Collection

AuthorisationCaseField/citizen.json

UserRoleCaseFieldIDCRUD
+citizen
+genApps
+CRUD

ComplexTypes/1_Party.json

FieldTypeParameterListElementCodeIDFieldTypeElementLabel
+
+id
+Party
+Text
+ 
+
+idamId
+Party
+Text
+ 

ComplexTypes/GeneralApplication.json

ElementLabelFieldTypeFieldTypeParameterIDListElementCode
+ 
+FixedRadioList
+GenAppType
+GeneralApplication
+applicationType
+ 
+Party
+GeneralApplication
+party
+ 
+DateTime
+GeneralApplication
+submittedOn
+ 
+DocumentWithId
+GeneralApplication
+submissionDocument
+ 
+Collection
+Document
+GeneralApplication
+supportingDocuments

ComplexTypes/2_DocumentWithId.json

ElementLabelFieldTypeIDListElementCode
+ 
+Text
+DocumentWithId
+id
+ 
+Document
+DocumentWithId
+document

Signed-off-by: sstewart <202802468+scottstewart-sl@users.noreply.github.com>
@scottstewart-sl scottstewart-sl changed the base branch from master to HDPI-4573_view_applications May 20, 2026 11:05
@scottstewart-sl scottstewart-sl marked this pull request as ready for review May 20, 2026 11:07
@scottstewart-sl scottstewart-sl requested review from a team as code owners May 20, 2026 11:07
@scottstewart-sl scottstewart-sl requested review from ayub-macalim, benouaer and guygrewal77 and removed request for a team May 20, 2026 11:07
@tvr-hmcts tvr-hmcts self-requested a review May 20, 2026 12:24
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.

Just some quesions from me.

@guygrewal77
Copy link
Copy Markdown
Contributor

Not related to your changes but is it possible to get some coverage on GenAppDocumentGenerator.getPartyLabel() to cover all scenarios ?

@scottstewart-sl
Copy link
Copy Markdown
Contributor Author

scottstewart-sl commented May 20, 2026

@guygrewal77

Not related to your changes but is it possible to get some coverage on GenAppDocumentGenerator.getPartyLabel() to cover all scenarios ?

I've actually refactored this code out in my next PR, (still in draft), and that adds coverage for that method, (after it was moved):

See DocumentNamingServiceTest.java in

https://github.com/hmcts/pcs-api/pull/1839/changes#diff-f85ea874cddd37a3d372bd2c30b1ef38c3ff4b26fd1237b503e45feb409d6916

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