Skip to content

Add "Meeting accessibility requirements" section and update guidance … - #1238

Merged
pooleycodes merged 2 commits into
mainfrom
2798--get-started
Jul 28, 2026
Merged

Add "Meeting accessibility requirements" section and update guidance …#1238
pooleycodes merged 2 commits into
mainfrom
2798--get-started

Conversation

@Jandums

@Jandums Jandums commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

…on data updates

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

Guidance updates: Add content on accessibility and PDFs and content around updating data

Related Tickets & Documents

QA Instructions, Screenshots, Recordings

Please replace this line with instructions on how to test your changes, a note
on the devices and browsers this has been tested on, as well as any relevant
images for UI changes.

Added/updated tests?

We encourage you to keep the code coverage percentage at 80% and above. Please refer to the Digital Land Testing Guidance for more information.

  • Yes
  • No, and this is why: please replace this line with details on why tests
    have not been included
  • I need help with writing tests

[optional] Are there any post deployment tasks we need to perform?

[optional] Are there any dependencies on other PRs or Work?

Summary by CodeRabbit

  • Documentation
    • Added guidance on how to handle previously correct versus incorrect records.
    • Added a new “Meeting accessibility requirements” section.
    • Clarified expectations for publishing planning data in open formats and planning documents as accessible PDFs.
    • Included links to accessibility and open-standards guidance.

@Jandums
Jandums requested a review from paris-dp July 24, 2026 12:42
@coderabbitai

coderabbitai Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e4fb2515-2cb3-4697-8320-96dd6eea4452

📥 Commits

Reviewing files that changed from the base of the PR and between 09a6f62 and 5fa6a19.

📒 Files selected for processing (1)
  • src/views/organisations/get-started.html
🚧 Files skipped from review as they are similar to previous changes (1)
  • src/views/organisations/get-started.html

Walkthrough

The organisation get-started page adds guidance for correcting existing records and a new section explaining accessibility requirements for published data and planning documents.

Changes

Get-started guidance

Layer / File(s) Summary
Update and accessibility guidance
src/views/organisations/get-started.html
The “Update your data” section now explains end dates and removal requests, while a linked “Meeting accessibility requirements” section adds CSV/GeoJSON and PDF guidance with relevant external links.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

Suggested reviewers: paris-dp

Poem

I’m a rabbit with guidance to share,
With open data made easy to bear.
Add end dates, send wrong records away,
Publish accessible files each day.
Hop through the contents, clear and bright!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title matches the main change: it adds an accessibility section and updates guidance.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 2798--get-started

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Jul 24, 2026

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 71.16% 2559 / 3596
🔵 Statements 70.37% 2701 / 3838
🔵 Functions 64.48% 512 / 794
🔵 Branches 64.26% 1268 / 1973
File CoverageNo changed files found.
Generated in workflow #1567 for commit 5fa6a19 by the Vitest Coverage Report Action

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/views/organisations/get-started.html`:
- Line 200: Update the open-formats link in the get-started page to use a live
section-specific destination without the stale `#open-formats` fragment, or remove
the fragment while preserving the existing guidance URL.
- Around line 197-202: Update the planning-documents bullet in the get-started
content so it does not present PDF as the sole required format; instruct
publishers to provide accessible HTML pages or an open format, using PDF only
where necessary alongside the accessible/open version. Keep the surrounding
open-formats guidance and links unchanged.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e621fd7f-444a-45e0-b9e2-5656874f5451

📥 Commits

Reviewing files that changed from the base of the PR and between 121a471 and 09a6f62.

📒 Files selected for processing (1)
  • src/views/organisations/get-started.html

Comment thread src/views/organisations/get-started.html
Comment thread src/views/organisations/get-started.html Outdated
paris-dp
paris-dp previously approved these changes Jul 27, 2026
Comment thread src/views/organisations/get-started.html Outdated
@pooleycodes
pooleycodes merged commit 9cc342f into main Jul 28, 2026
5 checks passed
@pooleycodes
pooleycodes deleted the 2798--get-started branch July 28, 2026 08:39
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