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

Feature/641 api feedback voting. Closes #641 #703

Merged
merged 24 commits into from
Dec 16, 2015

Conversation

jykae
Copy link
Contributor

@jykae jykae commented Dec 9, 2015

Basic functionality of feedback voting.

@55
Copy link
Contributor

55 commented Dec 9, 2015

Closes #641

'sumOfVotes': function () {
//Adds all vote integers together, returns sum of all votes as Integer
var feedbacks = FeedbackVotes.find({feedbackId: this._id}).fetch();
var votes = _(feedbacks).map(function(feedback){
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add line comment here. E.g.:

// Create list of all feedback vote values

@brylie
Copy link
Contributor

brylie commented Dec 10, 2015

Overall, great work @jykae!

Mainly, this is lacking line comments. There are also a few significant pieces to the feature, that are now better outlined in the definition of done.

@brylie
Copy link
Contributor

brylie commented Dec 10, 2015

Also, change the mouse cursor to a pointer mouse pointer when mouseover the up/downvote arrows.

@brylie brylie added the WIP label Dec 10, 2015
@brylie brylie self-assigned this Dec 10, 2015
@brylie brylie changed the title Feature/641 api feedback voting Feature/641 api feedback voting. Closes #641 Dec 10, 2015
@jykae jykae mentioned this pull request Dec 14, 2015
13 tasks
@brylie brylie removed the WIP label Dec 16, 2015
brylie added a commit that referenced this pull request Dec 16, 2015
Feature/641 api feedback voting. Closes #641

@jykae great work!
@brylie brylie merged commit d44bb50 into develop Dec 16, 2015
@brylie brylie deleted the feature/641-api-feedback-voting branch December 16, 2015 14:18
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

Successfully merging this pull request may close these issues.

None yet

3 participants