Skip to content

PO-2983#2288

Open
ChrisGarratt122 wants to merge 3 commits intomasterfrom
PO-2983
Open

PO-2983#2288
ChrisGarratt122 wants to merge 3 commits intomasterfrom
PO-2983

Conversation

@ChrisGarratt122
Copy link
Copy Markdown
Contributor

@ChrisGarratt122 ChrisGarratt122 commented Feb 26, 2026

Jira link

See PO-2983

Change description

  • Changed text from "You can continue with date in the past or change" to "You can continue with this date or change" in the case of user selecting a pay by date greater than 3 years in the future.

Testing done

  • Text updated when selecting a date greater than 3 years in the future, remains unchanged otherwise. Working as expected.

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

Copy link
Copy Markdown
Contributor

@louisbriggs louisbriggs 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