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

PC-550: Add the ability to add bespoke text to LAs on the outcome page #229

Merged
merged 8 commits into from
Jan 30, 2024

Conversation

CalPinSW
Copy link
Contributor

@CalPinSW CalPinSW commented Jan 5, 2024

https://beisdigital.atlassian.net/browse/PC-550

  • Use Partials in Pending and Confirmation Pages.
  • Add default Partials for existing copy.
  • Map LA Custodian Codes to Partial locations in QuestionaireController.
  • These changes had no effects on the unit tests. Feels like something we'd want integration/e2e tests for if we felt these changes warrant it. Not sure if we have any already? Let me know and I can add some.

…artials for existing copy. Map LA Custodian Codes to Partial locations in QuestionnaireController.
@CalPinSW CalPinSW changed the title PC-551: Add the ability to add bespoke text to LAs on the outcome page PC-550: Add the ability to add bespoke text to LAs on the outcome page Jan 8, 2024
HerPublicWebsite/Models/Questionnaire/PendingViewModel.cs Outdated Show resolved Hide resolved
{
return custodianCode switch
{
"2610" => "~/Views/Partials/ConfirmationLAMessage/Broadland.cshtml",
Copy link
Contributor

Choose a reason for hiding this comment

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

You haven't committed a file to this location. It looks like this should just be using the default, unless I'm misreading the ticket.

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, good point. Think I got ahead of myself and added this route before we had copy for it. I'll remove this case in the switch. Obviously we could remove the switch statement until we have copy for custom ones, but I think it's worth leaving in for future use unless your opposed to it?

HerPublicWebsite/Controllers/QuestionnaireController.cs Outdated Show resolved Hide resolved
HerPublicWebsite/Controllers/QuestionnaireController.cs Outdated Show resolved Hide resolved
HerPublicWebsite/Views/Questionnaire/Confirmation.cshtml Outdated Show resolved Hide resolved
Copy link
Collaborator

@Joe-Edwards Joe-Edwards left a comment

Choose a reason for hiding this comment

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

Thanks Calum, I think this is looking good now

@CalPinSW CalPinSW merged commit 947925b into develop Jan 30, 2024
1 check passed
@CalPinSW CalPinSW deleted the PC-550-la-bespoke-outcomes branch January 30, 2024 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants