Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add documentation using GitHub pages #229

Merged
merged 1 commit into from
Dec 19, 2024
Merged

Conversation

ocrease
Copy link
Collaborator

@ocrease ocrease commented Dec 18, 2024

@MindrustUK here is an initial PR to add documentation using github pages. You will need to configure pages, but that's easy to do. Go to https://github.com/MindrustUK/Heatmiser-for-home-assistant/settings/pages and point to the branch (dev?) once you've merged

@MindrustUK MindrustUK merged commit a1b3d3f into MindrustUK:dev Dec 19, 2024
2 checks passed
@ocrease
Copy link
Collaborator Author

ocrease commented Dec 19, 2024

@MindrustUK I see you configured github pages, but I think it's not quite right.

Source should be Deploy from a branch
Branch: dev
Folder: /docs

I suspect you have the folder set to / (root)?

Here is the site URL https://mindrustuk.github.io/Heatmiser-for-home-assistant/

@ocrease
Copy link
Collaborator Author

ocrease commented Dec 19, 2024

FYI, once configured correctly, it should look a bit like this: https://ocrease.github.io/Heatmiser-for-home-assistant/

@MindrustUK
Copy link
Owner

Odd, this was working! I'll investigate.

@ocrease
Copy link
Collaborator Author

ocrease commented Dec 20, 2024

@MindrustUK The Gihub Pages config on my fork looks like this:

image

In case you want to compare

@MindrustUK
Copy link
Owner

Hmmm, strange behavior. I had this working as you demonstrated above. Then when visited this morning on your prompt I got a 404, on a refresh I got what I suspect what you were seeing; effectively the readme.md, re-deployed twice. Same thing, switched to root, deployed switched back to dev deployed and it seems right now.

Is it looking right your end @ocrease ?

@ocrease
Copy link
Collaborator Author

ocrease commented Dec 20, 2024

@MindrustUK yes it looks good at the moment. We will have to update the main readme file to include a link to the docs page and maybe clean up any duplicate details that are not needed in the readme anymore

@ocrease ocrease deleted the Docs branch December 20, 2024 08:52
@MindrustUK
Copy link
Owner

@ocrease Agreed, thinking the same.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants