Skip to content

Update and add schemase - #877

Closed
TuukkaIkius wants to merge 11 commits into
mainfrom
TuukkaTahtinen/Schemas
Closed

TuukkaIkius wants to merge 11 commits into
mainfrom
TuukkaTahtinen/Schemas

Conversation

@TuukkaIkius

@TuukkaIkius TuukkaIkius commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

We had a ticket open on the marketing website board for updating and adding schemas on docs.
See linked asana ticket for more information https://app.asana.com/0/0/1218287037948702

Changes

  • Update the organization/corporation schema
    • Updated to match the organization schema displayed on goteleport.com/blog/
    • Is now of type organization and not type corporation, goteleport.com/ and goteleport.com/blog/ both use the organization type
  • Add TechArticle, HowTo and a BreadCrumb schema
    • BreadCrumb schema starts at goteleport.com as specified here
    • TechArticle and HowTo schemas currently require a publish date to be set in frontMatter for it to be included in the schemas
    • Last Modifed date uses the most recent changelog date as a final fallback
  • One of the specified requirements for full task completion is not addressed here: landing pages do not currently display the same type of schema as the pages under the landing page. Currently they default to the TechArticle schema.

On local dev all of the schema url:s resolve to host/page instead of host/docs/page but I assume (?) that is due to the pages themselves not resolving to a /docs/ path on local dev.

Schemas to confirm


@github-actions

github-actions Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

Amplify deployment status

Branch Commit Job ID Status Preview Updated (UTC)
TuukkaTahtinen/Schemas 67418b0 7 ✅SUCCEED tuukkatahtinen-schemas 2026-11-05 17:23:42

@ptgott

ptgott commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

Hi @TuukkaIkius, I know this is still in draft, but before opening this for review, would you mind adding an explanation of the motivation and context behind the PR to the description? Thanks!

@TuukkaIkius
TuukkaIkius marked this pull request as ready for review November 5, 2026 16:15
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-05T16:17:54.798770Z 676d3c6 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 676d3c6496

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread src/utils/structuredData.test.ts Resolved
@TuukkaIkius
TuukkaIkius deployed to docs-amplify November 5, 2026 16:46 — with GitHub Actions Active
@TuukkaIkius
TuukkaIkius marked this pull request as draft November 5, 2026 17:01

This branch was successfully deployed

1 active deployment
docs-amplify — 67418b06 Deployed Nov 5, 2026 by TuukkaIkius via Amplify Preview #2328
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.

2 participants