Modernize the application & Performance tweaks

[khaledosman]

The repo seems to be a bit out of date and isn't using best practices when it comes to performance for React applications, so this article might be unrealistic for 2019 https://medium.com/free-code-camp/a-realworld-comparison-of-front-end-frameworks-with-benchmarks-2019-update-4be0d3c78075. This PR aims to change that

• Used the newest versions of React and other dependencies
• Migrate away from deprecated library react-router-redux to connected-react-router
• Use PureComponents and React.memo for functional components for better performance
• Migrate away from deprecated lifeCycleHooks like componentWIllUpdate, componentWillReceiveProps & componentWillMount
• lazyload & codesplit non-default routes with React.lazy & React.Suspense

[sangdth]

I have the same feeling and looking forward to this PR.

[khaledosman]

Any updates?

[klamping]

FYI, using your fork, logging out gives me this error:

[khaledosman]

@klamping I was able to reproduce the issue, it should be fixed now.

[khaledosman]

Is anyone maintaining this app anymore? I would be happy to maintain it myself

[helderscrolls]

@khaledosman One small thing I found, currentUser.image is broken if you define no image, isn't there a way already coded on the API to recover the profile.image smiley face and use it instead of displaying the alt message ?

[khaledosman]

@helderscrolls Thanks, I'm not sure where that broke as I didn't change any API related logic, I'll look into it later when I have the time.

[helderscrolls]

@khaledosman I think it’s a issue that comes from the original Build, the only way I’ve found so far is to hard code the URL on the front-end, on the Header and the Comment preview that way even if currentUser.image is undefined it displays the smiley face