-
Notifications
You must be signed in to change notification settings - Fork 0
Feature login #1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Conversation
| "node-sass": "4.14.1", | ||
| "react": "^16.13.1", | ||
| "react-dom": "^16.13.1", | ||
| "react-scripts": "3.4.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd love to know why you removed react-scripts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't realize react script was removed, I simply ran npm install
UWEMAKAN
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests are mandatory. Please, write tests for your code.
I tested the build on my phone and it would look better if the inputs were aligned properly and the font sizes are the same as what's on the wireframe.
…t page, contact page, footer and navbar components.
I guys,
I made some changes like -> i installed react-router-dom, redux, react-redux, redux-thunk, font-awesome library, axios and i also finished the login and signup pages.