Skip to content
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

Comment counter #34

Merged
merged 8 commits into from
Nov 24, 2022
Merged

Comment counter #34

merged 8 commits into from
Nov 24, 2022

Conversation

V-Blaze
Copy link
Owner

@V-Blaze V-Blaze commented Nov 24, 2022

In this pull request, below are the Significant changes:

  • created CommentCounter.js file
  • Added functionality for counting item comments
  • Add functionality for updating DOM with correct comment count
  • Updated the readme file.

@V-Blaze V-Blaze linked an issue Nov 24, 2022 that may be closed by this pull request
Copy link
Collaborator

@usamamaliq usamamaliq left a comment

Choose a reason for hiding this comment

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

Hi @V-Blaze,

Changes Required♻️

Highlights 🥇

✔️ Following GitHub Flow
✔️ Descriptive PR Summary
✔️ Descriptive PR Title

14bo

Good job on implementing the comments counter. There are some issues that you need to work on before we can move forward

image

  • There are some items where when the name size increases all the pop-up section gets out of proportion. Kindly fix this to have a similar popup style for every item.

README.md Outdated
## 📝 License

- This project is [MIT](https://github.com/V-Blaze/FOODHUB.git/blob/main/LICENSE) licensed.
Copy link
Collaborator

Choose a reason for hiding this comment

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

image
The license file is not linked properly. Clicking the link gives an error. You can fix this by renaming the "LICENSE" file to "LICENSE.md" and updating the link in the readme as well.

Copy link
Collaborator

@usamamaliq usamamaliq left a comment

Choose a reason for hiding this comment

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

Hi @V-Blaze,

STATUS: APPROVED ✔️ ✔️

Your project is complete! There is nothing else to say other than... it's time to merge it
Congratulations and good job! 🎉
approved

Highlights

  • Good job fixing the required changes 👍

@V-Blaze V-Blaze merged commit 52983cf into dev Nov 24, 2022
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.

[3pt] - Add comments counter - Student B
2 participants