guyeilam / mernhood Goto Github PK

frontend/src/components/session/login_form.js

• When the active field is hovered over, it loses its green outline for a grey outline. Doesn't seem to be intentional.

• There should at least be a placeholder home page for logged in users that looks.. not empty.

./frontend/src/components/session/login_form.js

• No error message for failed authentication

Search API

The current implementation is not ideal. Instead the searchBar component should...

• Return (limit 10) possible search results from the given query so far
• Auto-highlight matching letters from the query of the search results
• Section query results into categories, i.e. companies and cypto as separate categories.

Reach goals

• Talk to stock price API to know whether the focused result is making or losing money, and should, therefore, be highlighted red vs green.

frontend/src/components/session/signup_form.js

• When the two passwords asked for in the form do not match up, no error is raised.

To do

• API key removed from project
• API key added to .gitignore

For consistency and general style, we should remove the word Component from the component names. Specifically, NavBarComponent should just be named NavBar.

frontend/src/components/nav/noUserNavLinks.js

• DropDown links "Components" to the login form. If this is on purpose, should display an error message asking the user to login before viewing components library.

frontend/src/components/session/signup_form.js

• Field currently being typed in is not being highlighted (our outlined) in green (like it is in the sign-in form)

• The project needs at the very least some instructions on how to run it.
• Next steps would be an overview of file structure
• Finally, some code examples would be good.

Somehow, we've ended up with two reset.css files. We need to merge these and delete one.

Should all the styles live inside the frontend/styles/ directory?

It looks like the main page component isn't loading after log in. The nav bar loads, but not the rest of the page.

The splash page navbar shouldn't be shown on the auth sign up and sign in pages.