Skip to content

Hot Module Reload doesn't seem to work #51

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

Closed
keithn opened this issue Aug 23, 2017 · 8 comments
Closed

Hot Module Reload doesn't seem to work #51

keithn opened this issue Aug 23, 2017 · 8 comments
Labels

Comments

@keithn
Copy link

keithn commented Aug 23, 2017

I just downloaded the repository, ran it, website came up fine, but any changes to files doesn't seem to update the website.

If I refresh the page, then the change appears

in the javascript console in chromeI have

EventSource's response has a MIME type ("text/html") that is not "text/event-stream". Aborting the connection.

Which keeps counting up.

( Using Visual Studio 2017 15.3.2 )

@MarkPieszak
Copy link
Member

MarkPieszak commented Aug 24, 2017

Yeah we're looking into this one, there's some kind of issue that appeared with dotnet 2.0 that's causing HMR to not really load properly!
Might be linked with aspnet/JavaScriptServices#1191

@keithn
Copy link
Author

keithn commented Aug 24, 2017

You've probably seen it.... but the pre packaged react project in visual studio 2017 has a HMR setup that works fine, I haven't had a look at what the differences are.

@MarkPieszak
Copy link
Member

Yeah, I need to see what might be causing it here. Might be something in StartUp.cs I'm missing / or a file like that.

@MarkPieszak
Copy link
Member

Might be that we need to get to 2.x of aspnet-webpack: "aspnet-webpack": "^2.0.1", can you give that a shot?

@keithn
Copy link
Author

keithn commented Aug 24, 2017

Yep, that worked.

@elucidsoft
Copy link
Contributor

Thank god I found this solution, I've spent 3 days trying to figure this out...Never thought of upping the version of aspnet-webpack oddly enough...Can you please make this change to the branch?

@MarkPieszak
Copy link
Member

MarkPieszak commented Aug 24, 2017

Sure, I'll make the change now!
Fixed via f578bdf

@canperk
Copy link

canperk commented Jan 7, 2018

Dude you saved my day :)

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

No branches or pull requests

4 participants