codephil-columbia / typephil

3 stars 1 forks source link

Change auth reducer to prevent 404s #40

Closed francesar closed 6 years ago

francesar commented 6 years ago

I've noticed that persistAuthConfig might not be be saving the isLoggedIn flag that app.js uses to decide whether to show the protected routes + their components due to the way that redux persist reconciles state during a rehydration (https://github.com/rt2zz/redux-persist#state-reconciler). Whitelisting currentUser and isLoggedIn separately seem to fix the issue Lmk what you think, since you've worked with persist more than I have

francesar commented 6 years ago

fix for #33

NWChen commented 6 years ago

good catch, thanks @francesar

sangjunp4rk commented 6 years ago

Also fixed https://github.com/codephil-columbia/typephil/issues/26