Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Added a method to update an existing rating. #4

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants

ags commented Feb 13, 2012

We needed to be able to update a user's existing rating for an item, so this adds that functionality.

Owner

diogenes commented Mar 23, 2012

Hey @ags! Sorry the delay and thanks for your contribution to Coletivo, man!

About the feature, it's a nice idea. Makes sense to me give the ability to update an existing rating.

But I think it can be simplified to be a concern of the existing Person#rate! method.
When the rating exists, updates the rating :weight, otherwise, create it. What do you think about?

Cheers,
Diogenes.

ags commented Mar 26, 2012

Sounds good, that was the original code but I didn't want to step on your toes :)

@ags ags closed this Oct 24, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment