Finished User Auth - Githubissues

wcsteve / swell

0 stars 1 forks source link

Finished User Auth #7

Open wcsteve opened 6 years ago

wcsteve commented 6 years ago

[ ] Backend: DB, model, controller, views
[ ] Redux Loop: ajax, actions, reducer
[ ] Presentational Components
[ ] Styling
[ ] Smooth, bug-free navigation
[ ] Adequate and appropriate seeds

ajansel commented 6 years ago

I think after these fixes you should be good to go!

Errors are hard to see, need to change the color at least
Need to clear errors between login / signin
Should probably remove the demo button from the nav bar and then just make sure that it's on both the signin and login form
When entering a random route it should redirect (to dashboard if logged in and splash page if not logged in) and update the route accordingly

wcsteve commented 6 years ago

I have fixed the errors issue. I think i like from a ux point of view to keep the demo button placed as is. I cannot figure out how to clear url when not logged in and accessing incorrect pages. The routes are protecting content, but not updating URL, could use some help...