Skip to content

Conversation

@sirwisdom
Copy link

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.

@UWEMAKAN UWEMAKAN requested review from MuhammadGb, Neyodhino, UWEMAKAN and Umoren and removed request for Neyodhino May 14, 2020 12:22
Thonze
Thonze previously approved these changes May 14, 2020
MuhammadGb
MuhammadGb previously approved these changes May 14, 2020
"node-sass": "4.14.1",
"react": "^16.13.1",
"react-dom": "^16.13.1",
"react-scripts": "3.4.1"
Copy link
Collaborator

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

Copy link
Author

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

Copy link
Collaborator

@UWEMAKAN UWEMAKAN left a 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.
@sirwisdom sirwisdom dismissed stale reviews from MuhammadGb and Thonze via 2e6aa83 May 19, 2020 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants