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

ci: update integration docs #2159

Merged
merged 5 commits into from
Dec 17, 2022
Merged

ci: update integration docs #2159

merged 5 commits into from
Dec 17, 2022

Conversation

astrobot-houston
Copy link
Contributor

This PR is auto-generated by a nightly GitHub action to update the individual integration pages from the integration READMEs in withastro/astro.

@astrobot-houston astrobot-houston added the ci Automated PRs created by CI, or PRs about automation - ask @delucis for help! label Dec 10, 2022
@netlify
Copy link

netlify bot commented Dec 10, 2022

Deploy Preview for astro-docs-2 ready!

Name Link
🔨 Latest commit be31729
🔍 Latest deploy log https://app.netlify.com/sites/astro-docs-2/deploys/639e295cf019ff000816f953
😎 Deploy Preview https://deploy-preview-2159--astro-docs-2.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 site settings.

Copy link
Member

@delucis delucis left a comment

Choose a reason for hiding this comment

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

Broken link that will need fixing at source in https://github.com/withastro/astro

The issue with the Spanish translation is a little trickier. It contains links to content in the MDX README which is no longer there after the revision. This content has now moved to the Markdown/MDX guide. But we can’t link to that because the Spanish document is the Markdown/MDX guide 😅

Best solution would probably be to ask if anyone is up for updating the guide translation.

src/pages/en/guides/integrations-guide/mdx.md Outdated Show resolved Hide resolved
@astrobot-houston astrobot-houston force-pushed the ci/docgen-integrations branch 5 times, most recently from d5c1a43 to b085c73 Compare December 15, 2022 12:04
delucis added a commit that referenced this pull request Dec 17, 2022
Removes outdated links from `pt-br` and `ru` translations that are causing errors in #2159 — these pages need updating so doing this in a separate PR so we can mark this to be ignored by the translation status tracker.
@delucis delucis merged commit 47c4153 into main Dec 17, 2022
@delucis delucis deleted the ci/docgen-integrations branch December 17, 2022 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Automated PRs created by CI, or PRs about automation - ask @delucis for help!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants