Merged
Conversation
|
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the Lung Cancer Screening “terms of use” template to improve formatting/wording, and applies a small Terraform locals formatting tweak.
Changes:
- Reformats the Introduction section markup and removes some superfluous phrasing in the terms of use template.
- Moves the “criminal offence” paragraph into the relevant list item and adjusts list markup in a few sections.
- Aligns
localsassignments in Terraform spoke variables.
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| lung_cancer_screening/questions/jinja2/terms_of_use.jinja | Reformats/adjusts HTML structure and wording for the terms of use page. |
| infrastructure/terraform/spoke/variables.tf | Minor whitespace/alignment change in the locals block. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -134,15 +135,15 @@ | |||
| <li>will always be available, error free, uninterrupted or free of viruses</li> | |||
| </ul> | |||
| <li>on any device or operating system that has been modified outside the mobile device or operating system vendor supported or warranted configurations. This includes devices that have been "jail-broken" or "rooted"</li> | ||
| <li>with someone else's NHS login account</li> | ||
| </ul> | ||
| <br> |
Comment on lines
+17
to
22
| We (NHS England) have developed a digital service called NHS check if you need a lung scan. | ||
| This is a digital way to access <a href="https://www.nhs.uk/tests-and-treatments/lung-cancer-screening/" target="_blank" rel="noopener">NHS lung cancer screening</a>. | ||
| This service is currently in pilot so is only available in certain areas, and if you are invited by your GP to take part. | ||
| <br /><br /> | ||
| <p>This service will not provide you with any results or information in relation to the lung cancer screening, you will need to complete your lung cancer screening by phone to receive a result.</p> | ||
|
|
mrlockstar
approved these changes
Mar 19, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What is the change?
Fixes to terms of use
Why are we making this change?
Terms of use properly formatted and not superfluous "use"