-
Notifications
You must be signed in to change notification settings - Fork 4
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
#161282285 Track if User has liked/disliked current comment #67
Conversation
Pull Request Test Coverage Report for Build 589
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
9879f60
to
cba5f06
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
- Fix comment like/dislike creation handler
cba5f06
to
91f6425
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How come you're reducing coverage 🤔
Please cover the uncovered lines.
- include relevant tests
*/ | ||
const hasReaction = (reactions, userId) => reactions.some( | ||
reaction => reaction.userId === userId | ||
); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected newline before ')' function-paren-newline
* | ||
* @returns {bool} retrurns true/false | ||
*/ | ||
const hasReaction = (reactions, userId) => reactions.some( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected newline after '(' function-paren-newline
@@ -0,0 +1,47 @@ | |||
import { expect } from 'chai'; | |||
import hasReaction from '../../utils/hasReaction'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing file extension for "../../utils/hasReaction" import/extensions
@@ -0,0 +1,47 @@ | |||
import { expect } from 'chai'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing file extension for "chai" import/extensions
@@ -3,6 +3,7 @@ import { | |||
Article, Comment, User, Reply, CommentLikesDislike, CommentHistory | |||
} from '../models'; | |||
import isEmpty from '../utils/is_empty'; | |||
import hasReaction from '../utils/hasReaction'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing file extension for "../utils/hasReaction" import/extensions
@waleCloud I have covered the uncovered lines. Thanks. |
What does this PR do?
Enable tracking user like/dislike state for each comment
Description of Task to be completed?
What are the relevant pivotal tracker stories?
#161282285