Skip to content

Conversation

@MosDevx
Copy link
Owner

@MosDevx MosDevx commented Dec 2, 2022

Create a module to Display user comments from Invlovement API
Create amodule to display a shows detail in full from the TvMaze API
Create a function to display number of comments visible on the screen

@MosDevx MosDevx changed the title Comments branch Comments branch Review Dec 2, 2022
@MosDevx MosDevx requested a review from aster-alemu December 2, 2022 20:58

**[Movie Api]** is a javascript project
**[Movie Api]** is a javascript project that fetches TV Show Data from an API

Copy link
Collaborator

Choose a reason for hiding this comment

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

Hi @MosDevx
Good job so far!

To Highlight! 👏 🟢

  • Good commit messages,
  • Great folder structure
  • Clean code
  • Good PR description
  • Your project is running free of Linter errors

There is some issue that you still need to work on but you are almost there!

Required Changes ♻️
Kindly add the link of live demo on your READEME file


We would like to thank Microverse

<p align="right">(<a href="#readme-top">back to top</a>)</p>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hi @MosDevx
Good job so far!

To Highlight! 👏 🟢

Good commit messages,
Great folder structure
Clean code
Good PR description
Your project is running free of Linter errors
There is some issue that you still need to work on but you are almost there!

Required Changes ♻️

  • Kindly remove the same element descriptions on your README file
  • Kindly add the link of your project live demo

@MosDevx MosDevx requested a review from aster-alemu December 2, 2022 22:58
Copy link
Collaborator

@aster-alemu aster-alemu left a comment

Choose a reason for hiding this comment

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

Status: APPROVED 🏆 🎉 🎊

  • Your project is complete

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants