Skip to content

docs: improve explanation for url config in GH Pages #6006

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

Merged
merged 1 commit into from
Nov 25, 2021

Conversation

Martinsos
Copy link
Contributor

Motivated by #6004 -> docs mention what to change when custom domain is used with github pages, but don't mention changing the url, which can be confusing (it was for me).

@netlify
Copy link

netlify bot commented Nov 25, 2021

✔️ [V2]
Built without sensitive environment variables

🔨 Explore the source changes: 13cbebb

🔍 Inspect the deploy log: https://app.netlify.com/sites/docusaurus-2/deploys/619f47b74b72460008920ff5

😎 Browse the preview: https://deploy-preview-6006--docusaurus-2.netlify.app

@github-actions
Copy link

⚡️ Lighthouse report for the changes in this PR:

Category Score
🟠 Performance 71
🟢 Accessibility 98
🟢 Best practices 100
🟢 SEO 100
🟢 PWA 95

Lighthouse ran on https://deploy-preview-6006--docusaurus-2.netlify.app/

Copy link
Collaborator

@Josh-Cena Josh-Cena left a comment

Choose a reason for hiding this comment

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

Great, makes sense 👍

@Josh-Cena Josh-Cena changed the title Update docs to mention setting url to custom domain for GH pages. docs: improve explanation for url config in GH Pages Nov 25, 2021
@Josh-Cena Josh-Cena added the pr: documentation This PR works on the website or other text documents in the repo. label Nov 25, 2021
@Josh-Cena Josh-Cena merged commit b1a4f39 into facebook:main Nov 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Signed Facebook CLA pr: documentation This PR works on the website or other text documents in the repo.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants