-
Notifications
You must be signed in to change notification settings - Fork 7
Move "people" under "OSPO" #86
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
base: main
Are you sure you want to change the base?
Move "people" under "OSPO" #86
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you think of a better title?