nss-evening-cohort-11 / WildflowerCoffeeGifts

0 stars 2 forks source link

Auth works #101

Closed Nikababy01 closed 3 years ago

Nikababy01 commented 3 years ago

How to Test: First go to Firebase project for WildFlowerCoffee and click on the authentication tab and set up your email and password image Next

  1. Run Visual Studio's and npm start to get the website
  2. When site loads you should only see the Home Page, scroll down and should only see the heading for product themes and not data, click the products tab and you should not see the products
  3. Next click the Log In tab and login using the email and password, this should return you to the home page and if you scroll down, you should see the themes, next click on the products page and you should see all products.
ancasimon commented 3 years ago

Monique, here are a couple of things that it may be good to change:

  1. The Need to register link doesn't seem to be working - should we remove it? OR comment it out for now until we get the registration feature working?
  2. Once I log in, I no longer see the Home page - see screenshot below - but I assume we should still have access to it? UPDATE: I just realized that Beth might have removed the Home page before this change - and replaced it with just clicking the logo. So you can ignore this bullet @Nikababy01 !

And a couple of suggestions that don't have to be fixed right now - but that we should not forget about:

  1. If I click Log In again > it still shows me the username and password fields (filled in this time) - but I wonder if we can hide them once the user is logged in? This can be done in a later ticket too if it seems too much right - just create another ticket if you think that would be better.
  2. We should also have a ticket for finalizing the navbar based on authentication - for example, everyone - even if not logged in - should have access to the Home and Products and Log In tabs - but only logged in users should have access to Orders and the Single view pages. Not sure about the Shopping Cart - we can discuss

no_home_page_if_logged_in