mithi/hello-tiny-box

feat: Add PWA support and improve lighthouse performance

mithi opened this issue · 9 comments

mithi commented

Screen Shot 2020-09-01 at 9 18 00 PM

arjita-mitra commented

Hello, @mithi I would like to contribute. I am a first-time contributor so please guide me. I have good experience in front end especially react and javascript. I have worked on lighthouse performance in the past. :-)

arjita-mitra commented

@mithi is there any contribution guideline for your project? Please share.

mithi commented

@mithi is there any contribution guideline for your project? Please share.

You can follow the guidelines here: https://github.com/mithi/hexapod/blob/master/CONTRIBUTING.md

Thank you! :)

mithi commented
arjita-mitra commented

Sure @mithi .

  • 😄1
arjita-mitra commented

Hello @mithi
Can you assign this issue to me? I have started working on it.

mithi commented
arjita-mitra commented

Hello @mithi
It's been a while I am stuck at this issue, because of this I am unable to test service worker also. Service workers can be tested in production mode only. So I tried to deploy it in a static server following this-
https://create-react-app.dev/docs/deployment/
https://create-react-app.dev/docs/making-a-progressive-web-app/

I am getting an error, Uncaught SyntaxError: Unexpected token '<'. I read multiple articles to resolve this issue, but no luck so far. So far I understood its some issue with the homepage url in package.json. This message is important -

The project was built assuming it is hosted at /hello-tiny-box/.
You can control this with the homepage field in your package.json.

Below is some reference regarding this issue.

webpack/webpack#2882 (comment)
https://stackoverflow.com/questions/54340240/create-react-app-build-uncaught-syntaxerror-unexpected-token
https://stackoverflow.com/questions/51071848/blank-screen-for-express-react-app-at-production-uncaught-syntaxerror-unexpect

Any help would be appreciated.