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
Milestone 1 : Initialize project with components (React only). #2
Conversation
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 @Chrissiku,
Your project is complete! There is nothing else to say other than... it's time to merge it
Congratulations! π
Cheers and Happy coding!πππ
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.
src/components/homePage/Book.js
Outdated
<> | ||
<div className="book"> | ||
<div className="left-book"> | ||
<span className="type">Action</span> | ||
<h3>The Hunger Games</h3> | ||
<span className="author">Suzanne Collins</span> | ||
<ul> | ||
<li> | ||
<a>Comments</a> | ||
</li> | ||
<li> | ||
<a>Remove</a> | ||
</li> | ||
<li> | ||
<a>Edit</a> | ||
</li> | ||
</ul> | ||
</div> | ||
|
||
<div className="right-book"> | ||
<div className="progress"> | ||
<div className="circle"> | ||
<div className="inner"></div> |
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 make sure that the book component has the two required props
title
andauthor
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.
π’ Project Approved! π
Hi @Chrissiku π
Your project is complete! βοΈ π
Finally, you did it! π π. Your code was very professional π― and I can see you have a great chance to be a Great Software Engineer βοΈ.
There is nothing else to say other than... it's time to merge it
Congratulations! π. And Welcome to Microverse π.
Note: You can now take a break even if short before you move on to the next project. Happy π Coding π₯οΈ! and Have a nice day π.
Highlights π
- βοΈ Descriptive commmit messsages
- βοΈ Descriptive PR description
- βοΈ No linter Errors
- The code is top-notch! π― and have a huge potential.
Cheers and Happy coding!πππ
Important Notes π
- Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
- Please, remember to tag me in your question so I can receive the notification.
Thank you for your time π and call me if you need help or clarification βοΈ
As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.
Thank you @OmarRamoun for the approval of the project |
What I add to the project π»
What I use π¨βπ»
What I implement β»οΈ
<Route>
s and<Link>
s for the app's navigationTechnology used π§βπ»
Reactjs
Babel
Project New Overview