-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feature to add a comment #27
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 @yonas44 great work 👍 , It was so close
STATUS: CHANGES REQUIRED ♻️
Here are some things you need to work on:
- Take a look at the comment below
- Kindly make sure that the comment show up un the screen when it gets added, because that is included in the project requirement
OPTIONAL:
Those are optional and you can skip them:
- Kindly give the form elements some padding, so it look nice on the screen
If you have any questions please feel free to ask 👍
Happy coding 💯
src/api/addComment.js
Outdated
@@ -0,0 +1,27 @@ | |||
const addComment = async (id, name, insight) => { | |||
try { | |||
const res = await fetch(`${process.env.INVOLVMENT_URL}/apps/${process.env.APP_ID}`, { |
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.
Please make sure that the URL is correct because the addComment is failing with status code 404
Thank you @Abdessittir for your feedback, I have added the changes please have a look at it and give me your feedback! |
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 @yonas44 amazing work 💯, there is nothing to say other than it's time to merge your pull request.
STATUS: APPROVED 🥇
Highlights 💪🏻
✅ Comment posted to the API correctly
✅ No linter errors
✅ Dom manipulated to add new comment 💯
Happy Coding 👍
Hello @Abdessittir, so in this feature I implemented comment adding functionality, when you add a comment;