Skip to content

PPHA-677: Accessibility statement#395

Merged
jamiefalcus merged 3 commits intomainfrom
PPHA-677-accessibility-statement-footer
Mar 23, 2026
Merged

PPHA-677: Accessibility statement#395
jamiefalcus merged 3 commits intomainfrom
PPHA-677-accessibility-statement-footer

Conversation

@jamiefalcus
Copy link
Copy Markdown
Contributor

What is the change?

Adding the accessibility statement

Why are we making this change?

Allowing users to view our accessibility statement

Copilot AI review requested due to automatic review settings March 23, 2026 15:31
@jamiefalcus jamiefalcus force-pushed the PPHA-677-accessibility-statement-footer branch from a1422ee to f2d0753 Compare March 23, 2026 15:31
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds an accessibility statement page to the lung cancer screening questionnaire site and makes it discoverable via the global footer, with feature coverage to confirm the page is reachable both logged-in and logged-out.

Changes:

  • Add a new /accessibility-statement route backed by a Jinja template.
  • Add an “Accessibility statement” link to the global footer.
  • Extend static-page feature coverage to include the new page and navigation via footer links.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
lung_cancer_screening/questions/urls.py Adds a new URL pattern for the accessibility statement page.
lung_cancer_screening/questions/jinja2/accessibility_statement.jinja Introduces the accessibility statement content template.
lung_cancer_screening/core/jinja2/layout.jinja Adds footer navigation link to the new statement page.
features/static.feature Updates BDD coverage to assert accessibility statement availability and navigation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread lung_cancer_screening/questions/jinja2/accessibility_statement.jinja Outdated
Comment thread lung_cancer_screening/questions/jinja2/accessibility_statement.jinja Outdated
Comment thread lung_cancer_screening/questions/jinja2/accessibility_statement.jinja Outdated
@jamiefalcus jamiefalcus force-pushed the PPHA-677-accessibility-statement-footer branch from fc47fd3 to f657485 Compare March 23, 2026 16:00
Copy link
Copy Markdown
Contributor

@stephhou stephhou left a comment

Choose a reason for hiding this comment

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

Checks failed

@stephhou stephhou dismissed their stale review March 23, 2026 16:32

Checks passing

@jamiefalcus jamiefalcus force-pushed the PPHA-677-accessibility-statement-footer branch from 2e24ab0 to 434e710 Compare March 23, 2026 16:38
@sonarqubecloud
Copy link
Copy Markdown

@jamiefalcus jamiefalcus merged commit f90bb18 into main Mar 23, 2026
25 checks passed
@jamiefalcus jamiefalcus deleted the PPHA-677-accessibility-statement-footer branch March 23, 2026 16:52
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