Skip to content

Update troubleshooting-custom-domains-and-github-pages.md#4957

Merged
lucascosti merged 3 commits intogithub:mainfrom
JOJ0:patch-4
Apr 1, 2021
Merged

Update troubleshooting-custom-domains-and-github-pages.md#4957
lucascosti merged 3 commits intogithub:mainfrom
JOJ0:patch-4

Conversation

@JOJ0
Copy link
Copy Markdown
Contributor

@JOJ0 JOJ0 commented Mar 31, 2021

Why:

The current docs were misleading to me and I had to ask github support why it doesn't work as expected. Support agent Tammy Metz was very helpful and clarified that actually only the special subdomain www is redirected to the apex domain.

What's being changed:

Clarify that using both apex and custom subdomain is not generally unsupported and redirects between www and apex domain are automatically set up by github.

I wrote "If configured correctly" - there should be a link a place where this is explained. Running out of time this moment but could add this later if you point me in the right direction. Thanks in advance!

@github-actions github-actions Bot added the triage Do not begin working on this issue until triaged by the team label Mar 31, 2021
tmetz
tmetz previously approved these changes Mar 31, 2021
Copy link
Copy Markdown
Contributor

@tmetz tmetz 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 taking the time to submit these PRs, @JOJ0! I definitely think this clarifies exactly what the confusion was. Looks good to me, so now we'll see if the Docs team has any suggestions!

@janiceilene janiceilene added pages Content related to GitHub Pages waiting for review Issue/PR is waiting for a writer's review and removed triage Do not begin working on this issue until triaged by the team labels Mar 31, 2021
@janiceilene
Copy link
Copy Markdown
Contributor

@JOJ0 Thank you so much for opening this PRs 😍 I've triaged this for the team to take a look ✨

@lucascosti lucascosti self-assigned this Apr 1, 2021
Copy link
Copy Markdown
Contributor

@lucascosti lucascosti left a comment

Choose a reason for hiding this comment

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

I made a small change, and this is ready to go!

Thanks so much for all these contributions, @JOJ0! 🙇‍♂️

@lucascosti lucascosti enabled auto-merge (squash) April 1, 2021 03:15
@lucascosti lucascosti merged commit 238f982 into github:main Apr 1, 2021
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 1, 2021

Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. If you're looking for your next contribution, check out our help wanted issues

@JOJ0 JOJ0 deleted the patch-4 branch April 2, 2021 05:21
@JOJ0
Copy link
Copy Markdown
Contributor Author

JOJ0 commented Apr 2, 2021

thumbs up and many thanks! :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pages Content related to GitHub Pages waiting for review Issue/PR is waiting for a writer's review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants