-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
[i18nIgnore] Integration READMEs in docs! #5839
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
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.
Very cool! Was confused about this whole process for quite some time :P
Co-authored-by: Voxel <voxelmc@hotmail.com>
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.
Awesome job @sarah11918, just a few a suggestions 😁
src/content/docs/zh-cn/guides/integrations-guide/cloudflare.mdx
Outdated
Show resolved
Hide resolved
Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>
Amazing work catching these, @dreyfus92 ! 🚀 |
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.
I'm not sure that we can delete the script yet, but everything else is looking good!
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.
This is great!
Co-authored-by: Reuben Tier <64310361+TheOtterlord@users.noreply.github.com>
Co-authored-by: Voxel <voxelmc@hotmail.com> Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com> Co-authored-by: Reuben Tier <64310361+TheOtterlord@users.noreply.github.com> Co-authored-by: = <otterlord.dev@gmail.com>
Description (required)
This performs all the
content
changes required for the Integration READMEs :/en/
page has updated frontmatter:- No more dev warning
-
githubURL
changed togithubIntegrationURL
to separate logic forEditLink.astro
andPageTitle.astro
(Error messages and config-ref still need to usegithubURL
for off-site editing, but the integrations need a header link back to the package on GitHub, too.-
hasREADME: true
deleted/scripts/generate-integration-pages.ts
so that this file is included. Will need Chris to review what changes (deletion?) needs to be made here.EditLink.astro
ended up with an extra blank line, but at least it's here for easy reference in case Chris decides he prefers to tackle this from that end, vs my strategy to updatePageTitle.astro
logic.Tested behaviour:
Accompanying tasks outside of docs code:
remove CI task
Merge corresponding
withastro/astro
andwithastro/adapters
PRs:Make current docs content changes corresponding to Not yet merged Solid feature
Make sure no new PRs are opened to integration/adapter READMES