shellfly
shellfly
@datatalking Glad to know that this repo is helpful to you. I like the idea to supply context or industry usage for each algorithm. I'm not quite sure what's "(semi-universal)...
@harikvpy Hi, thanks for considering making contribution back to this repo, and I can see the swappable feature could be useful in some cases. However this change introduces a third...
@apetryla you can use `vote_count` and `vote_exists` in template now, the README is also updated to add example code.
It seems the `vote_vote.object` filed in the database becomes a `varchar` type which should be an integer. Did you do any database migration recently? could you double-check the table schema...
@y2kbadbug If you use uuid for object_id, then where does this numeric ID come from ? `"vote_vote"."object_id" = 15663665..` what does your `quote` table schema looks like now?
@y2kbadbug it seems a valid solution to change the field type to be compatible with the content object, I just checked the Django document which suggests the same solution. do...
@Barthelemy, thanks for the PR. Based on the [document](https://docs.djangoproject.com/en/5.1/releases/4.2/#index-together-option-is-deprecated-in-favor-of-indexes), it needs a migration to rename the index. Could you add it as well? Also, please update the [CI config](https://github.com/shellfly/django-vote/blob/master/.github/workflows/ci.yml#L14) to...
> I have added the migration that Django generated. I am not sure why there is the `AlterField` though. It's probably related to [this change](https://docs.djangoproject.com/en/5.2/releases/3.2/#customizing-type-of-auto-created-primary-keys). Could you set DEFAULT_AUTO_FIELD before...
@Barthelemy, please go ahead and update the supported versions. Users can always use the previous tag version if needed.
@Barthelemy, I won't have time to update this shortly, but pull requests are welcome