docs icon indicating copy to clipboard operation
docs copied to clipboard

Add content - configure subdomain for github pages site that was built using actions workflow

Open brianwaithaka opened this issue 1 year ago • 4 comments

Why:

Current docs do not cover use case where using a static generator and a custom GitHub Actions workflow builds and publishes directly to github-pages environment

Since workflow builds and deploys directly, adding a cname file only covers use case of deploying GitHub Pages from a branch

https://docs.github.com/en/pages/configuring-a-custom-domain-for-your-github-pages-site/managing-a-custom-domain-for-your-github-pages-site#configuring-a-subdomain

Closes: #26014

What's being changed (if available, include any code snippets, screenshots, or gifs):

https://docs.github.com/en/pages/configuring-a-custom-domain-for-your-github-pages-site/managing-a-custom-domain-for-your-github-pages-site#configuring-a-subdomain

  • Changing this doc by adding a new entry under point 8
  • Adding a reusables.pages.build-remotely-verify-subdomain file with the actual text
Screenshot 2023-06-15 at 10 19 52

Check off the following:

  • [x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline.

    • For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.
  • [x] For content changes, I have completed the self-review checklist.

brianwaithaka avatar Jun 15 '23 15:06 brianwaithaka

Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

welcome[bot] avatar Jun 15 '23 15:06 welcome[bot]

Automatically generated comment ℹ️

This comment is automatically generated and will be overwritten every time changes are committed to this branch.

The table contains an overview of files in the content directory that have been changed in this pull request. It's provided to make it easy to review your changes on the staging site. Please note that changes to the data directory will not show up in this table.


Content directory changes

You may find it useful to copy this table into the pull request summary. There you can edit it to share links to important articles or changes and to give a high-level overview of how the changes in your pull request support the overall goals of the pull request.

Source Preview Production What Changed
pages/configuring-a-custom-domain-for-your-github-pages-site/managing-a-custom-domain-for-your-github-pages-site.md fpt
ghec
fpt
ghec

fpt: Free, Pro, Team ghec: GitHub Enterprise Cloud ghes: GitHub Enterprise Server ghae: GitHub AE

github-actions[bot] avatar Jun 15 '23 15:06 github-actions[bot]

@brianwaithaka Thanks so much for opening a PR! We appreciate the extra context and screenshot! ✨

I'll get this triaged for review :zap:

In the meantime, if you're looking for other contributions, check out our help wanted section to find open issues you can work on. 👍

cmwilson21 avatar Jun 16 '23 19:06 cmwilson21

Thanks for opening a pull request! We've triaged this issue for technical review by a subject matter expert :eyes:

github-actions[bot] avatar Jun 26 '23 14:06 github-actions[bot]

@brianwaithaka Checked in with the team on this one, this is a good point but the PR is somewhat incorrect. Point 8 in the doc page linked above does not apply to sites built with a custom workflow. Instead of storing the domain in a CNAME file, it needs to be done in the Pages settings. Linking to the domain verification doc is a good general call out but it does not describe how to associate a custom domain with a site, only how to verify a domain.

Can you update the PR to reflect the team feedback?

cmwilson21 avatar Jul 31 '23 21:07 cmwilson21

👋 Forgot to mention, ping me here in the issue when the updates have been made and I'll get the review process started on it again ✨

cmwilson21 avatar Aug 10 '23 20:08 cmwilson21

Well noted, I will update accordingly

brianwaithaka avatar Aug 12 '23 02:08 brianwaithaka

Hi @cmwilson21 , sorry for the delay. Just some clarification needed please.

So issue was that when building the site and publishing directly to gh-pages env + setting up the subdomain in pages settings, i still could not get it to work. Docs only suggest cname file method which applies to deploying from a branch

Therefore, by verifying the subdomain under account settings, after setting it up in pages settings for the repo, it worked. Do you suggest this info be provided as a note or additional point under 8 in the #configuring-a-subdomain section ? on this site

Please guide since team feedback suggests scraping the point from here altogether and/or perhaps placing it under the troubleshooting page

brianwaithaka avatar Aug 24 '23 02:08 brianwaithaka

@brianwaithaka Thanks for following up with this! I think making it a note will do. The reviewer may have other thoughts but by committing that update, we can get it on the review board again 👀

cmwilson21 avatar Aug 30 '23 18:08 cmwilson21

Hello, @cmwilson21 , updates made. Please review

brianwaithaka avatar Sep 14 '23 15:09 brianwaithaka

@brianwaithaka Sorry for the delay! Some folks from the team have been out so I've had to ask in other places about this one. Unfortunately, I'm super unfamiliar with this subject, but the last SME I checked in with left this feedback - "I would refactor step 8 into the verification they mentioned, since it applies to both cases, and pull out step 8 and 9 into options, since neither are steps you need to always take"

cmwilson21 avatar Oct 03 '23 13:10 cmwilson21

This PR has been automatically closed because there has been no response to to our request for more information from the original author. Please reach out if you have the information we requested, or open a new issue to describing your changes. Then we can begin the review process.

github-actions[bot] avatar Oct 25 '23 19:10 github-actions[bot]