Skip to content

Conversation

@MosDevx
Copy link
Owner

@MosDevx MosDevx commented Dec 3, 2022

Kindly Review My Project

  1. Created a project with web-pack
  2. Interacted with TV-maze API and Involvement API
  3. Implemented Single Page Application Functionality
  4. Displayed list of fetched items on page load
  5. Displayed details of chosen item upon click
  6. Implement functionality to add and record likes
  7. Implement functionality to display and add new comments on a given item
  8. Added test with Jest framework

Copy link

@BregornOriginal BregornOriginal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Team 🟡 🟡 🟡

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!

one-step-closer-one-more

Great work doing this capstone project, it looks excellent. Congratulations on this, but there's one little change to do before having the approve 🥇

Suggested changes

Check the comments under the review.

You can use as many of my suggestions as you want. If there is anything you would like to skip - feel free to do that. However, I strongly recommend you to 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.

@@ -0,0 +1,3 @@
const showsNumber = (list) => list.length;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Kindly make sure to add a function to show the items on the home page.

image

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.

4 participants