Closed MatthewL246 closed 9 months ago
Yeah I also do not have secrets access :( so lets not merge it right now
I looked at the fork though and i do really like that change, so I think it is a needed one
Alright, I think this is pretty much completed. We just need to set up Cloudflare and the secrets before merging.
@MatthewL246 Maybe email roche today. I have the password for our cloud flare account if you need it
@michaell4438 Done. Did you reserve the Pages project domain?
Alright, the Cloudflare project is ready: xbhs-robotics-docs.pages.dev. I'll create the API token when necessary.
Your docs changes were built successfully! View a preview of them at https://f382469e.xbhs-robotics-docs.pages.dev
@michaell4438 can i haz approval
Changes:
I created a job for the GitHub Pages action that uploads a test deployment to Cloudflare Pages. I tested this on my fork using my own Cloudflare account, see an example PR here: https://github.com/MatthewL246/FtcRobotController/pull/7.
Controls and Usage:
When you are making a documentation update, make sure to name your branch starting with
docs/
. After the build completes, you will receive a comment from the GitHub Actions bot on your PR with a link to a preview of the deployment.Required steps before merging:
xbhs-robotics-docs.pages.dev
.CLOUDFLARE_ACCOUNT_ID
with the account ID.CLOUDFLARE_API_TOKEN
with an API token that has edit access toAccount.Cloudflare Pages
.Reviewers:
@XaverianTeamRobotics/senior-programmers
Closes #593.