django-vote icon indicating copy to clipboard operation
django-vote copied to clipboard

Issue 71: separated count votes functions

Open edegeyer opened this issue 5 years ago • 3 comments

#71 implementation of the suggested changes

edegeyer avatar Aug 03 '20 07:08 edegeyer

@edegeyer The changes break backward compatibility and also introduce duplicate code. I would like to do it like this:

def count_up(self):
    return self.count(self, UP)
def count_down(self):
    return self.count(self, DOWN)

Also, at first this project doesn't support down vote, that's why the default counts up actions. The README is not updated, do you think mention it in document is better than add two extra methods?

shellfly avatar Aug 03 '20 08:08 shellfly

Mentioning this in the README would definitely help, because for example I wasn't aware about it, as the action "DOWN" occurs and Downvotes can be stored/made. For me personally, it would be more useful, if Downvoting is fully supported and would be happy to help with the steps towards a full support of that function

edegeyer avatar Aug 03 '20 09:08 edegeyer

@edegeyer There are two fields num_vote_up and num_vote_down added when implementing down vote. They can be used to replace the count api now. If you're willing to update the PR to improve the README, that would be great.

shellfly avatar Aug 03 '20 13:08 shellfly