-
Notifications
You must be signed in to change notification settings - Fork 280
DOC-5993 more metadata fields #2434
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?
Conversation
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.
My comments, FWIW (which isn't much these days). I don't fully grasp why we need this, but maybe that doesn't matter. One thing for sure: the relatedPages lists will be a nightmare to maintain. We do tend to migrate content quite a bit in these docs, so this will be a lot of extra and, IMHO, unnecessary effort. That said, despite my misgivings, I'll approve.
This adds a way to pass metadata through from the page source frontmatter to the published page (only select fields, not the whole lot). I've also added some of this metadata to a few pages to get started. Eventually, we should probably implement this more widely.
BTW, there's nothing directly visible in the pages (or at least there shouldn't be!) but you can see the metadata in the
index.html.mdversion of the page, if you're interested.