cds-snc / node-starter-app

Quick start application setup.... because you have to start somewhere.
MIT License
5 stars 3 forks source link

[WIP] Browser-sync and Webpack Notifier integration #126

Closed dsamojlenko closed 4 years ago

dsamojlenko commented 4 years ago

Can't figure out why this isn't working the way it should - when using the expressjs integration (see app.js changes), when you run npm run dev it will open a browser tab as expected. However, when a change is observed, instead of refreshing that tab, it will open a new tab.

Note that if you disable the express integration and run npm run dev in one console tab and npm run sync in another, everything works as expected.

I tried doing this same thing in Webpack instead of Express, and same result. I can provide the code for that integration, if that would be the preferred place for integration.

timarney commented 4 years ago

Closing this - hopefully to be re-visted at some point.