-
Notifications
You must be signed in to change notification settings - Fork 0
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
Capstone Project: TvMaze #7
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 @miltonHenschel ,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
- You have submitted a PR that does not have your code changes to be reviewed
- [ ] No video presentation
- [ ] No deployment link
- [ ] No Acknowledgment as per project requirement
Such a Pull Request can not be accepted for review.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
Thanks for the remarks @CollinsTatang. Actually, this is the second time I'm submitting this. Please kindly checkout the suggestion from my last code reviewer: #5. |
Hi @miltonHenschel Don't forget to add the old PR link to your explanation. |
Thanks once more @CollinsTatang. I did just that. 🙏 |
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 @miltonHenschel,
Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!
Highlights
- No linter errors ✔️
- Correct GitFlow ✔️
- Descriptive
README
file ✔️ - Descriptive
Commit
history ✔️
Required Changes ♻️
- [REQUIRED] Good job so far! You should record a
video
presenting the features of the project and the video contains aclear
andobjective
description of the project in professional English, thetopic
andpurpose
of the project are made clear, thedemo
of the project,highlighting
all the project features andhighlight
about a piece of the code the student finds interesting or is proud of making but I couldn't find your presentation video, please implement it 🙏🏼
- [REQUIRED] Good job so far! You should have
unit
tests for several components of theapp
and some purefunctions
in the Redux code also App hasintegration
tests using React Testing Library. You should perform a mocking data fetching and Redux Store connection 🙏🏼
- [REQUIRED] Good job so far! You should follow the design guidelines including correctly:
- Typography: font
face
,size
,weight
,UI
- Layout:
composition
andspace
between elements.
Your Design
Behance Design
Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better.
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.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
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.
Hey @mukhammadrizooff, which mockup site helped you to come up with those beautiful pictures of site please? |
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 @miltonHenschel ,
STATUS: APPROVED 🎉
In my opinion, your project is ready for the presentation! There is nothing else to say other than... it's time to merge it
Good luck! 🎉
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Wow! Thanks very much @christianbiring1. I didn't think I woulda succeed this early. I only used 2/5 reviews. If we could accumulate the number of reviews left. 😆 🙏 |
Select the API you are going to use.
Build files structure for your React app.
Prepare routes and navigation in your app.
Make sure that a user can display a list of items and filter them. Data should come from the API.
Create the tests for the application.
Style your components to match the design provided.
Deploy the project and test for final details.
Record a video for your project.
Create a good README and PR description.