Conversation
samcunliffe
commented
Dec 17, 2025
- Resolves Move people under the OSPO section on the website #62.
- Also fix some breadcrumbs now you've taught me how to do it.
There was a problem hiding this comment.
Pull request overview
This pull request reorganizes the site structure by moving the "people" page under the OSPO section and fixes breadcrumb navigation across multiple OSPO pages. The changes improve the information architecture by grouping related content together and correcting previously self-referential breadcrumbs.
Key changes:
- Removed the
/people/permalink to relocate the people page under/ospo/people - Fixed breadcrumb navigation on three OSPO pages to properly link to the parent OSPO section
- Added short titles to pages for improved navigation metadata
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| ospo/people.md | Removed /people/ permalink and updated breadcrumb to point to OSPO parent; added short title |
| ospo/talks-proposals.md | Fixed self-referential breadcrumb to point to OSPO parent instead; added short title |
| ospo/business-case.md | Fixed self-referential breadcrumb to point to OSPO parent instead; added short title |
Note: While not part of this PR's scope, there are existing links to /people/ in index.md (line 9) and ospo/index.md (line 18) that will need to be updated separately to point to the new location at /ospo/people.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| short: "Who are we?" | ||
| --- | ||
|
|
||
| # Who are we? |
There was a problem hiding this comment.
Can you think of a better title?
This comment was marked as outdated.
This comment was marked as outdated.
|
@dpshelio did you change your mind about this? |
|
No, I hadn't changed mind, just got too centred on the business case and forgot this. |
There was a problem hiding this comment.
🚫 [vale] reported by reviewdog 🐶
[proselint.Annotations] 'TODO' left in text.
There was a problem hiding this comment.
🚫 [vale] reported by reviewdog 🐶
[proselint.Annotations] 'TODO' left in text.