search

hatchways / team-cherry

1 stars 1 forks source link

Bug: Header doesn't rerender after logout #62

Open jt5301 opened 4 years ago

jt5301 commented 4 years ago

The search bar and settings button still show up when the user clicks logout

grogsy commented 4 years ago

When clicking logout, should I be redirecting the user instead of doing history.push('/login')?

grogsy commented 4 years ago

I've checked out this bug, and what I'm seeing happen is that on line 136 of Header.js {getUser() ? (... getUser() doesn't seem to get called, which suggests to me at least that the Header component isn't being rerendered after signing out

grogsy commented 4 years ago

Forgot to mention, the way I figured is by placing a console.log on the Header component right before rendering in an attempt to inspect the value returned by getUser(). It didn't get called iirc