Skip to content
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

Update the links on a few pages to point to the new website repo. #201

Merged
merged 1 commit into from
Nov 5, 2023

Conversation

Julian
Copy link
Member

@Julian Julian commented Nov 3, 2023

Fixes a few 404s, specifically on the Glossary page.

This does not update the pages/blog/posts/website-analytics-snapshot-2023.md 2023 blog post (which directly links to the json-schema-org.github.io source repository and references behavior that isn't the case for the new website). json-schema-org/community#514 is relevant here, and should likely consider what to do with this link and blog text as part of that PR.

This also does not touch files in the _include directory, which are submodules included from the spec repo and will have their own PR there.

Fixes a few 404s, specifically on the Glossary page.

This does not update the pages/blog/posts/website-analytics-snapshot-2023.md
2023 blog post (which directly links to the json-schema-org.github.io
source repository and references behavior that isn't the case for the
new website). json-schema-org/community#514 is relevant here, and should
likely consider what to do with this link and blog text as part of that
PR.

This also does not touch files in the _include directory, which are
submodules included from the spec repo and will have their own PR there.
Copy link

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: ef6be62
Status: ✅  Deploy successful!
Preview URL: https://d3275902.website-2v2.pages.dev
Branch Preview URL: https://update-website-links.website-2v2.pages.dev

View logs

Copy link
Collaborator

@benjagm benjagm left a comment

Choose a reason for hiding this comment

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

Thanks for adding this Julian.

@Julian Julian merged commit 8f388e4 into main Nov 5, 2023
@Julian Julian deleted the update-website-links branch November 5, 2023 03:04
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