Skip to content
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

Using update_at field instead of create_at #65

Open
YemreGurses opened this issue Oct 7, 2019 · 3 comments
Open

Using update_at field instead of create_at #65

YemreGurses opened this issue Oct 7, 2019 · 3 comments

Comments

@YemreGurses
Copy link

Hi,
If a user changes his/her vote, it's create time does not change. Is it more sense to use update_at instead of create_at?

@Gilbishkosma
Copy link

Hi @YemreGurses , create_at is used to store the DateTime when the user first time voted.
And update_at will be changed each time the user change his/her vote.
Both create_at and update_at has their own features.
And i think it is good to use update_at and create_at separately.

@YemreGurses
Copy link
Author

Yea, thats better. My point was storing changed vote time and django-vote does not have that feature afaik.

@shellfly
Copy link
Owner

@YemreGurses In the beginning, this project doesn't support down vote, so one's vote record either exists with a created_at or been deleted. Since version 2.1.4 which down vote was supported, that's when one can update his/her vote record. So yes, that's one feature that can be added to this project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants