Skip to content
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

Live refresh/Hot reload #627

Closed
kMutagene opened this issue Jan 6, 2021 · 7 comments
Closed

Live refresh/Hot reload #627

kMutagene opened this issue Jan 6, 2021 · 7 comments

Comments

@kMutagene
Copy link
Contributor

Currently, when developing docs via fsdocs watch, one has to manually reload the browser page after the new files are generated. As this project uses Suave, it should be possible to mirror the changes of this PR to fornax, which implemented live refresh there.

@dsyme
Copy link
Contributor

dsyme commented Jan 6, 2021

Sounds good

Kevin, I think I should make you Co admin on this repot and nuget (I'm on vacation and haven't republished the nuget yet)

@dsyme
Copy link
Contributor

dsyme commented Jan 6, 2021

Kevin, what's your nuget id please?

@kMutagene
Copy link
Contributor Author

I'll be glad to help where I can, sorry to have bothered you during vacation, enjoy the rest of it ;). My nuget profile is here: https://www.nuget.org/profiles/Mutagene.

@dsyme
Copy link
Contributor

dsyme commented Jan 11, 2021

@kMutagene I've sent the request on nuget

@kMutagene
Copy link
Contributor Author

@dsyme thank you! I was just writing the release docs for the next package (feels a little strange to write those about commits that are not my own) and wondering about #621 and how that commit affects the version number of the next package, will it be 8.0 or 7.3? When that is sorted i will release the package today

@dsyme
Copy link
Contributor

dsyme commented Jan 13, 2021

@kMutagene When in doubt bump the major version :-)

dsyme added a commit that referenced this issue Jan 13, 2021
* Add server side hot reload message cycling for the watch command

* Add substitution-based injection of hot reload script into templates

* add note for watch script to styling docs

* Incorporate review-requested changes

Co-authored-by: Don Syme <[email protected]>
@kMutagene
Copy link
Contributor Author

Closed as of #629

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

No branches or pull requests

2 participants