#161966615 like and dislike articles #25
Conversation
![codeclimate[bot]](https://avatars.githubusercontent.com/in/9403?s=60&v=4){kind=small}
| return Response({'message':"article not found."}) | ||
|
|
||
|
|
||
| class LikeArticle(UpdateAPIView): |
There was a problem hiding this comment.
Similar blocks of code found in 2 locations. Consider refactoring.
| return Response(message, status.HTTP_200_OK) | ||
|
|
||
|
|
||
| class DislikeArticle(UpdateAPIView): |
There was a problem hiding this comment.
Similar blocks of code found in 2 locations. Consider refactoring.
| body = serializers.CharField() | ||
| author = UserSerializer(read_only = True) | ||
|
|
||
| def likes(self, instance): |
There was a problem hiding this comment.
Similar blocks of code found in 2 locations. Consider refactoring.
| liked = instance.likes.all().filter(id=user_id).count() == 1 | ||
| return {'likes': instance.likes.count(), 'User': liked} | ||
|
|
||
| def dislikes(self, instance): |
There was a problem hiding this comment.
Similar blocks of code found in 2 locations. Consider refactoring.
salma-nyagaka
force-pushed
the
ft-like-dislike-articles-161966615
branch
5 times, most recently
from
0b6ee09
to
d6c0e4b
Compare
salma-nyagaka
requested review from
SnyderMbishai,
verenceLola,
actlikewill,
SilasKenneth,
TeamoreA and
Mnickii
salma-nyagaka
force-pushed
the
ft-like-dislike-articles-161966615
branch
2 times, most recently
from
cdfdca6
to
fb8c508
Compare
| user = request.user | ||
| # checks for the boolean value of liking an article | ||
| liked = bool(user in article.like.all()) | ||
| if liked is True: |
There was a problem hiding this comment.
Similar blocks of code found in 2 locations. Consider refactoring.
| user = request.user | ||
| # checks for the boolean value of disliking an article | ||
| disliked = bool(user in article.dislike.all()) | ||
| if disliked is True: |
There was a problem hiding this comment.
Similar blocks of code found in 2 locations. Consider refactoring.
salma-nyagaka
force-pushed
the
ft-like-dislike-articles-161966615
branch
3 times, most recently
from
b901d6a
to
80747b0
Compare
…sliking an article - add tests for liking and disliking fuctionality - modify the base file to accomodate the tests - add fields in the model for liking and disliking an article - add a method in the serializer for returning a liked or disliked functionality - add views for liking and disliking an article. [Delivers #161966615]
SilasKenneth
force-pushed
the
ft-like-dislike-articles-161966615
branch
from
80747b0
to
e4dcd39
Compare
|
Code Climate has analyzed commit e4dcd39 and detected 4 issues on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
What does this pull request do?
This PR implements the functionality of liking and disliking articles.
Description of the tasks to be completed?
PUT /api/slug/like/PUT /api/slug/dislike/How should this be manually tested?
testing:
Screenshots:
Like an article
Unlike an article
Disliked an article
Un-dislike an article
What are the relevant Pivotal Tracker Stories
#161966615
Checklist: