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

Pin guidelines/* content when publishing 2.2 docs to w3.org #4161

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

kfranqueiro
Copy link
Contributor

@kfranqueiro kfranqueiro commented Dec 4, 2024

This applies the same publication-version-pinning logic added in #4007 for 2.1 docs, to 2.2 docs specifically when built for w3.org, so that guideline/SC and definition text included within informative docs matches the published recommendation. Local dev and gh-pages builds continue to run against guidelines in the local repository (effectively the Editor's Draft).

The build uses the published recommendation whenever WCAG_VERSION is specified, so enabling this for w3.org builds only requires adding that environment variable explicitly to the publish-w3c script.

The changes to 11ty/guidelines.ts fix cases that were never exhibited in the 2.1 build, but came up when testing building 2.2 against the published recommendation.

@kfranqueiro kfranqueiro requested a review from iadawn December 4, 2024 21:15
Copy link

netlify bot commented Dec 4, 2024

Deploy Preview for wcag2 ready!

Name Link
🔨 Latest commit e448e70
🔍 Latest deploy log https://app.netlify.com/sites/wcag2/deploys/67531f29c41cc500083f7861
😎 Deploy Preview https://deploy-preview-4161--wcag2.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 configuration.

@w3cbot
Copy link

w3cbot commented Dec 4, 2024

kfranqueiro marked as non substantive for IPR from ash-nazg.

@kfranqueiro kfranqueiro force-pushed the kgf-pin-22-guidelines branch from b7f246b to e448e70 Compare December 6, 2024 15:58
@kfranqueiro kfranqueiro merged commit d3385ba into main Dec 19, 2024
5 checks passed
@kfranqueiro kfranqueiro deleted the kgf-pin-22-guidelines branch December 19, 2024 17:27
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.

3 participants