Skip to content

Adds docs on how to enable Live Reload in a starter app #252

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

Merged
merged 2 commits into from
Nov 17, 2023

Conversation

cy-by
Copy link
Contributor

@cy-by cy-by commented Nov 15, 2023

#183429068

This feature is really a configuration change in IntelliJ. Code wise, I documented the steps and then removed some application configuration that isn't needed as it wasn't contributing to allowing Live Reload to work.

…Live Reload

- Add docs on how to set up Live Reload
@enyia21 enyia21 self-assigned this Nov 16, 2023
Copy link
Contributor

@enyia21 enyia21 left a comment

Choose a reason for hiding this comment

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

There is only one change that I'd like to see made

README.md Outdated
* [Firefox extension](https://addons.mozilla.org/en-US/firefox/addon/livereload-web-extension/?utm_source=addons.mozilla.org&utm_medium=referral&utm_content=search)
* [Chrome extension](https://chromewebstore.google.com/detail/livereload/jnihajbhpnppcggbcgedagnkighmdlei)
* Restart your browser after install
* Go to `http://localhost:8080/`
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we alert developers that they need to start up their applications before visiting http://localhost:8080?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I moved around the structure so that it makes more sense, to your point. Thank you!

- Add chromium browser list
Copy link
Contributor

@enyia21 enyia21 left a comment

Choose a reason for hiding this comment

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

Great

@enyia21 enyia21 merged commit d304be7 into main Nov 17, 2023
@enyia21 enyia21 deleted the enable-live-reload-#183429068 branch November 17, 2023 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants