-
Notifications
You must be signed in to change notification settings - Fork 3
Final-Capstone-Front-end #17
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: main
Are you sure you want to change the base?
Conversation
Set user authentication
Authentication login signup
Add workingreservestyle
Style the book section
Meltrust
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.
Hi @Irirwanirira ,
Good job so far!
There are some issues that you still need to work on to prepare your project for the final evaluation, but you are almost there!
To highlight:
- Nice code organization ✔️
- App is working well✔️
- Good readme ✔️
- Nice frontend implementation✔️
You are really close to finishing the Microverse program!! Keep it up! 👍👍👍
After implementing the requested changes, please submit another review request. ♻️
Check the comments under the review.
Cheers and Happy coding!👏👏👏
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the previous reviews unless it is requested otherwise.
| import Input from './reusable/inputFields/input'; | ||
| import isUserSigned from '../../helpers/auth'; | ||
|
|
||
| const AddBook = () => { |
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.
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.
done.
README.md
Outdated
| ### Run tests | ||
|
|
||
| To run tests, run the following command: | ||
|
|
||
| ```sh | ||
| rspec spec/ | ||
| ``` |
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.
- This command is meant for the backend project. Since this is the frontend project, kindly remove it. 👍
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.
Omitted accordingly.
| /* .backdrop { | ||
| display: flex; | ||
| flex-direction: column; | ||
| } | ||
| */ |
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.
- Kindly, remove this commented code in order to improve the repo's cleanliness. 👍
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.
Removed.
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.
- Kindly, remove the commented code present in this file in order to improve the repo's cleanliness. 👍
Meltrust
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.
Hi @Irirwanirira,
Good job so far!
There are some issues that you still need to work on to prepare your project for the final evaluation, but you are almost there!
To highlight:
- Nice code organization ✔️
- App is working well✔️
- Good readme ✔️
- Beautiful frontend ✔️
You are really close to finishing the Microverse program!! Keep it up! 👍👍👍
After implementing the requested changes, please submit another review request. ♻️
Check the comments under the review.
Cheers and Happy coding!👏👏👏
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the previous reviews unless it is requested otherwise.
|
|
||
| <p align="right">(<a href="#readme-top">back to top</a>)</p> | ||
|
|
||
| <!-- SUPPORT --> |
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.
- Since the last review, this pull request has not suffered any changes or commits. Kindly implement the comments featured on the last review. 👍
✅ Deploy Preview for fastidious-biscotti-61dbed ready!
To edit notification comments on pull requests, go to your Netlify site settings. |


This Pr request consists of:
Link to the Front end Pull Request
Thank you for Reviewing our project