Skip to content

HDPI-5164-QA#947

Open
madhavi1203 wants to merge 10 commits intoHDPI-5164-remove-session-dependency-in-respond-to-claimfrom
HDPI-5164-QA
Open

HDPI-5164-QA#947
madhavi1203 wants to merge 10 commits intoHDPI-5164-remove-session-dependency-in-respond-to-claimfrom
HDPI-5164-QA

Conversation

@madhavi1203
Copy link
Copy Markdown
Contributor

@madhavi1203 madhavi1203 commented Mar 12, 2026

Jira link

https://tools.hmcts.net/jira/browse/HDPI-5164

Change description

Dynamic data update Remove session storage from respond to claim journey

Testing done

image

NOTES:
Failed tc is due to regression bug introduced on Dev PR, Dev is going to fix it.

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

@madhavi1203 madhavi1203 changed the base branch from master to HDPI-5164-remove-session-dependency-in-respond-to-claim March 16, 2026 13:21
@madhavi1203 madhavi1203 added enable_full_functional_tests enable_all_page_functional_tests Enables page functional tests for all e2e pages. labels Mar 16, 2026
@madhavi1203 madhavi1203 marked this pull request as ready for review March 17, 2026 15:49
@madhavi1203 madhavi1203 requested review from a team as code owners March 17, 2026 15:49
@madhavi1203 madhavi1203 requested review from adusumillipraveen and scottstewart-sl and removed request for a team March 17, 2026 15:49
getmainHeader: (claimantName: string): string => {
return `Have you paid any money to ${claimantName} since 16th June 2025?`;
},
respondToClaimParagraph: 'Respond to a property possession claim',
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could you please use backticks (`) from lines 5 to 14?

Copy link
Copy Markdown
Contributor

@SameenaHMCTS SameenaHMCTS left a comment

Choose a reason for hiding this comment

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

Could you please update your branch as all functional tests have been moved to respective pft files. so you might have to redo these changes of src/test/ui/functional/respondToAClaim.spec.ts

# Conflicts:
#	src/test/ui/data/page-data/noticeDateWhenNotProvided.page.data.ts
#	src/test/ui/data/page-data/repaymentsMade.page.data.ts
#	src/test/ui/functional/respondToAClaim.spec.ts
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