Skip to content
This repository has been archived by the owner on Sep 8, 2021. It is now read-only.

Update React 16.12 & utilize hooks #144

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

bahtou
Copy link

@bahtou bahtou commented Nov 21, 2019

covers PR 140
covers PR 142
covers PR 95
covers PR 68

will use this as a basis to update the rest of the package versions (react-router, redux etc).
will also update redux to use hooks

* transform all class components to function components
* transform lifecycle methods to hooks
* destructure props for all components
* add `defaultState` to reducers
* create new component `UserInfo`
* miscellaneous: newlines, spaces, consistent return
@ezorfa
Copy link

ezorfa commented Aug 28, 2020

Hi @bahtou It looks like there is an error when i sign in:

Screenshot 2020-08-28 at 09 41 24

and in the console:

Screenshot 2020-08-28 at 09 42 03

Do you have any solution for this error? I am using django backend

@djimmo
Copy link

djimmo commented Oct 20, 2020

@ezorfa Are you sure you:

  1. checked out branch ver-class-updates of Bahtou's fork?
  2. removed node_modules/ and package-lock.json and ran npm install again for a clean install of dependencies
  3. cleared you browser's localStorage (where it saves a jwt token)

Doing these 3 things should fix your issue :)

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

Successfully merging this pull request may close these issues.

3 participants