jeffposnick / create-react-pwa

https://github.com/facebookincubator/create-react-app + Progressive Web App goodness

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Compatibility with latest create-react-app dependencies

camel113 opened this issue · comments

I am not sure to understand this sentence in your readme.
When running a production server locally, make sure you use a different port than 3000, to ensure the service worker does not inadvertantly take control of the development environment.

Can we test service workers locally (with classic method npm start) with your changes?

I think your changes do not work with "react-scripts": "0.7.0",

Thanks for the heads-up. I haven't had a chance to test this out with react-scripts 0.7.0 yet, and it's entirely possible that the setup needs to be modified. I'll keep this issue open to track that.

(I need a little time to get to it, though.)

I have react-scripts 0.7.0 as well, and looks like the manifest is getting served with the mime type as text/html. Is there any work around for this?

I've updated things with an example of using the latest create-react-app dependency.

Can you take a look at c-r-a-0.6.0...c-r-pwa-0.6.0 and confirm that making those changes to something scaffolded with the latest create-react-app works for you?

I've heard from other folks that have confirmed it's working now. Please reopen this issue if you're still seeing issues with react-scripts 0.7.0.

Thank you @jeffposnick I do not find the time to check it but I will do it soon ;-) As you said other folks confirmed so I trust them ;-)