Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Recurring payment terms and conditions page #1854

Merged
merged 9 commits into from
Jan 24, 2024

Conversation

ScottDormand96
Copy link
Collaborator

https://eaflood.atlassian.net/browse/IWTF-3667

Terms and conditions page for recurring payments

@ScottDormand96 ScottDormand96 added the enhancement New feature or request label Jan 11, 2024
Copy link
Collaborator

@jaucourt jaucourt left a comment

Choose a reason for hiding this comment

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

Good work - alter the spelling if you like, but it's too small to request changes, so I'll leave it to your discretion!

])('back button tests for $handlerName page', ({ pageHandler }) => {
it.each([[CONTROLLER.uri], [RECURRING_TERMS_CONDITIONS.uri]])(
'addLanguageCodeToUri is called with %s when referrer is %s',
async referer => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

You've spelt referrer correctly in the description, but wrong here. I almost let you off because it's [misspelt in the http spec](https://privacyinternational.org/guide-step/4150/http-referer-explanation#:~:text=What%3A%20the%20Referer%20(a%20misspelling,request%20received%20by%20a%20server.), but it jars too much, sorry!

@@ -283,7 +326,7 @@ describe('guidance page handlers', () => {
})
})

const getMockRequest = (i18nValues, search = '') => {
const getMockRequest = (i18nValues, referer) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

It's ok here because it leads to more concise code!

Copy link

@ScottDormand96 ScottDormand96 merged commit 1998b2a into develop Jan 24, 2024
3 checks passed
@ScottDormand96 ScottDormand96 deleted the feature/iwtf-3667-rp-terms-and-conditions branch January 24, 2024 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants