-
Notifications
You must be signed in to change notification settings - Fork 2
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
8 return just number of likes #20
base: master
Are you sure you want to change the base?
8 return just number of likes #20
Conversation
@franeklubi There is solution on these issue. Please check them. Thanks |
Yes I will eventually, sorry for the long wait There's not enough time on my side |
@franeklubi it's OK this pull? |
I don't have time to check, You'll need to wait a little |
Sure i understand. :-) it's OK |
hallo it's OK ? I need know it this week because i will make presentation about my work on this issue in to the school. |
@franeklubi Hi, please can you check my pull request? |
Hi, I've checked Your pull request, and there are a couple of issues. Firstly, You need to do the same for comments. I didn't write that explicitly, and thought that that was clear, so I'm sorry for omitting that requirement in the original issue. Now the issue is amended. Secondly, you don't actually need to fetch the users on the front end, because there's no view to show the users who liked the post, so the fetch is unnecessary. Sorry for the long wait and sorry for omitting these points in the original issue description. |
@franeklubi in the first task you didn't write something about comments only just post . You can make new task for comments and i can do on it. |
ce1ac63
to
a71dadb
Compare
@franeklubi Hi, Likes on posts and comments are done and fetching the users on the front end was removed. Please let's check this soon as possible because i need know it this week |
@franeklubi Hallo, please can you check my pull request? |
No description provided.