Skip to content

Fix MEP-8 identity crisis - #340

Merged
vknabel merged 1 commit into
metal-stack:mainfrom
GeertJohan:fix/mep-8-frontmatter
Aug 12, 2026
Merged

Fix MEP-8 identity crisis#340
vknabel merged 1 commit into
metal-stack:mainfrom
GeertJohan:fix/mep-8-frontmatter

Conversation

@GeertJohan

@GeertJohan GeertJohan commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

The MEP-8 page (Configurable Filesystemlayout) had slug, title, and sidebar position referring to MEP-7, which is actually a different, declined proposal that only exists as a pull request.

Added a redirect to preserve SEO.

@GeertJohan
GeertJohan requested a review from a team as a code owner August 8, 2026 13:11
@metal-robot metal-robot Bot added the area: documentation Affects the documentation area. label Aug 8, 2026
@metal-robot metal-robot Bot added this to Development Aug 8, 2026
@netlify

netlify Bot commented Aug 8, 2026

Copy link
Copy Markdown

Deploy Preview for metal-stack-io ready!

Name Link
🔨 Latest commit 6dc06ce
🔍 Latest deploy log https://app.netlify.com/projects/metal-stack-io/deploys/6a7b58f03e58a300095ea801
😎 Deploy Preview https://deploy-preview-340--metal-stack-io.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment thread docusaurus.config.ts Outdated
},
],
[
"@docusaurus/plugin-client-redirects",

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.

I think the redirects are managed here: https://github.com/metal-stack/website/blob/main/static/_redirects

But @vknabel is the master of this universe.

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.

Correct! We rely on Netlify redirects as they are on HTTP level instead of being client side.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ahh great! Will fix!

@GeertJohan
GeertJohan force-pushed the fix/mep-8-frontmatter branch from 7ba9e71 to 829a3c4 Compare August 11, 2026 17:14
The MEP-8 page (Configurable Filesystemlayout) had slug, title, and
sidebar position referring to MEP-7, which is actually a different,
declined proposal that only exists as a pull request.

A Netlify redirect keeps external links to the old
/community/MEP-7-configurable-filesystem-layout-for-machine-allocation
URL working.
@GeertJohan
GeertJohan force-pushed the fix/mep-8-frontmatter branch from 829a3c4 to 6dc06ce Compare August 11, 2026 17:16

@vknabel vknabel 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.

Great! Thanks for the contribution and taking care of it!

@vknabel
vknabel merged commit bb1c902 into metal-stack:main Aug 12, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: documentation Affects the documentation area.

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

4 participants