Skip to content

Conversation

@FaisalMoJ
Copy link
Contributor

Change description

Applicant2 address question pages.

JIRA link (if applicable)

https://tools.hmcts.net/jira/browse/NFDIV-4922

Before merging a pull request make sure that:

  • tests have been updated / new tests has been added (if needed)
  • README and other documentation has been updated / added (if needed)

If this ticket will have any visible impact on users and is not behind a feature toggle, make sure that:

  • this ticket been reviewed by QA
  • the user story been signed off by the PO

Note that bug fixes, dependency updates and technical tasks do not directly impact the user experience and can be merged without QA and PO review.

If this user story cannot be immediately merged find a way to put it behind a feature toggle and get it merged.

Comment on lines 7 to 22
title: 'It’s easier to proceed with a postal address',
line1:
'It’s usually faster and less expensive if you can provide us with a postal address. If you wish to proceed without an address, you will need to complete and pay for an additional application once you have submitted this one.',
line2: `If you cannot find your ${partner}’s address, you may also provide your ${partner}’s:`,
solicitorAdderss: 'solicitor’s address, if they have one',
workAddress: `work address, if you have your ${partner}’s permission`,
line3:
'You could also try looking for them on social media, or contacting their friends or relatives if you feel safe doing so.',
line4: 'You can save your application here and try to find an address if you need to.',
foundTheirAddress: `Have you been able to find your ${partner}’s address?`,
errors: {
applicant1FoundApplicant2Address: {
required,
},
},
});
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like we might need to add the bullet points from the design and remove formatting. Design:
image

Frontend:
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah sorry i will fix this now

Comment on lines 1 to 20
import { Applicant2Represented } from '../../../app/case/definition';
import { TranslationFn } from '../../../app/controller/GetController';
import { FormContent } from '../../../app/form/Form';
import { isFieldFilledIn } from '../../../app/form/validation';
import { CommonContent } from '../../common/common.content';

const en = ({ isDivorce, partner }: CommonContent) => ({
title: `Does your ${partner} have a solicitor representing them?`,
line1: `The court only needs to know if your ${partner} has a solicitor who is representing them for
${isDivorce ? 'the divorce' : 'the process to end your civil partnership'}.
You do not need to tell the court about a solicitor they use for other purposes.`,
yes: 'Yes, they have a solicitor representing them',
no: 'No, they do not have a solicitor representing them',
notSure: "I'm not sure",
errors: {
applicant1IsApplicant2Represented: {
required: 'You have not answered the question. Select an answer before continuing.',
},
},
});
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like we may be able to remove this page? I can't see where it's being used

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes i pushed this change by mistake i will remove that

@@ -0,0 +1,57 @@
import config from 'config';
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: I don't think the URL path used in the UCD prototype is a very good description for this page. It's currently set as:
"address-what-you-need"

Maybe we could adjust it? Fubara didn't mind when we changed URLs for No Response:
"how-to-progress-without-an-address"

line2: `You’ll be able to do this once you’ve submitted your ${
isDivorce ? 'divorce application' : 'application to end civil partnership'
}.`,
line3: `This will cost £58, but you may be able to <a class="govuk-link" target="_blank" href="${config.get(
Copy link
Contributor

Choose a reason for hiding this comment

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

We could make the £58 fee dynamic/load it from fees config

FaisalMoJ and others added 2 commits November 20, 2025 16:13
Co-authored-by: adamg-hmcts <159780146+adamg-hmcts@users.noreply.github.com>
Comment on lines 376 to 377
if (data.applicant1IsApplicant2Represented === Applicant2Represented.YES) {
return data.applicant1KnowsApplicant2Address === YesOrNo.YES ? ENTER_THEIR_ADDRESS : ADDRESS_FINDING;
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it right that we only run this logic when the applicant thinks app2 is represented?

Copy link
Contributor

Choose a reason for hiding this comment

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

It looks as though the else if block below may need to be removed, as the NEED_TO_GET_ADDRESS page doesn't exist anymore in the new journey. We could delete that page and some of the others that are being removed

@hmcts-jenkins-j-to-z
Copy link
Contributor

Plan Result (aat)

Plan: 0 to add, 1 to change, 0 to destroy.
  • Update
    • module.nfdiv-frontend-redis6.azurerm_redis_cache.redis
Change Result (Click me)
  # module.nfdiv-frontend-redis6.azurerm_redis_cache.redis will be updated in-place
  ~ resource "azurerm_redis_cache" "redis" {
        id                                 = "/subscriptions/1c4f0704-a29e-403d-b719-b90c34ef14c9/resourceGroups/nfdiv-cache-aat/providers/Microsoft.Cache/redis/nfdiv-frontend-aat"
        name                               = "nfdiv-frontend-aat"
        tags                               = {
            "application"         = "no-fault-divorce"
            "autoShutdown"        = "true"
            "builtFrom"           = "https://github.com/HMCTS/nfdiv-frontend.git"
            "businessArea"        = "CFT"
            "contactSlackChannel" = "#no-fault-divorce"
            "environment"         = "staging"
            "managedBy"           = "No Fault Divorce"
        }
        # (24 unchanged attributes hidden)

      ~ redis_configuration {
          ~ maxfragmentationmemory_reserved         = 125 -> 642
          ~ maxmemory_delta                         = 125 -> 642
          ~ maxmemory_reserved                      = 125 -> 642
            # (14 unchanged attributes hidden)
        }
    }

Plan: 0 to add, 1 to change, 0 to destroy.

@hmcts-jenkins-j-to-z
Copy link
Contributor

Plan Result (prod)

No changes. Your infrastructure matches the configuration.

Comment on lines +8 to +23
title: `${
isApplicant2Represented ? `Your ${partner}’s postal address` : `Do you have your ${partner}'s postal address?`
}`,
line1: `${
isApplicant2Represented
? `We need your ${partner}’s address so that we can notify them about the ${
isDivorce ? 'divorce' : 'application to end civil partnership'
} by post.`
: hasAContactForPartner
? `The court needs your ${partner}'s address, to notify them by letter about the ${
isDivorce ? 'divorce' : 'ending your civil partnership'
}.`
: `It’s important you provide your ${partner}'s address. You did not provide their email address and the court needs a way to ‘serve’ (deliver) the ${
isDivorce ? 'divorce' : 'ending your civil partnership'
} papers to them.`
}`,
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this conditional content from Fubara? I had the impression they'll now see the same content regardless of whether applicant 2 is represented

Comment on lines +376 to 380
if (data.applicant1KnowsApplicant2Address === YesOrNo.NO) {
return ADDRESS_FINDING;
} else {
return ENTER_THEIR_ADDRESS;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: We could make this a ternary expression like the other paths, for consistency and because they're slightly easier to read

Co-authored-by: adamg-hmcts <159780146+adamg-hmcts@users.noreply.github.com>
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.

2 participants