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

Fix various reloading issues in debug and production modes #53

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

maximmold
Copy link
Contributor

Fix hotswapping of LESS files by ensuring the file path is returned from the ServletContextResourceReaderHandler. There was only one resourceInfoProvider which is a BaseServletContextResourceReader and it does not implement ResourceGenerator

Fix issues with child bundles being included instead of parent bundle because parent bundled was being used.
Ensure config change event doesn't occur on first check of configChanged.

Stop listeners and watchers and create anew after config change to ensure the bundlesHandler is consistent between the resource watcher and the rebuilding of dirty bundles in JawrRequestHandler

…rom the ServletContextResourceReaderHandler. There was only one resourceInfoProvider which is a BaseServletContextResourceReader and it does not implement ResourceGenerator

Fix issues with child bundles being included instead of parent bundle because parent bundleId was being used.
Ensure config change event doesn't occur on first check of configChanged.

Stop listeners and watchers and create anew after config change to ensure the bundlesHandler is consistent between the resource watcher and the rebuilding of dirty bundles in JawrRequestHandler
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.

1 participant