-
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
#167703750 change on authorisation #71
Conversation
#165413102 User should be able to logout
…into develop pull origin develop
…into develop pull origin develop
helpers/commentHelper.js
Outdated
currentComment.dataValues.histories = commentHistories; | ||
return currentComment; | ||
}) | ||
); |
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
helpers/commentHelper.js
Outdated
currentComment.dataValues.histories = commentHistories; | ||
return currentComment; | ||
})); | ||
await Promise.all( |
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
controllers/article.controller.js
Outdated
const numberOfSharesOnPlatform = await ArticleHelper.numberOfSharesOnPlatform(shares); | ||
const numberOfSharesOnPlatform = await ArticleHelper.numberOfSharesOnPlatform( | ||
shares | ||
); |
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
controllers/article.controller.js
Outdated
@@ -238,7 +238,9 @@ class ArticleController { | |||
.status(200) | |||
.send({ message: 'this article has not been shared yet' }); | |||
} | |||
const numberOfSharesOnPlatform = await ArticleHelper.numberOfSharesOnPlatform(shares); | |||
const numberOfSharesOnPlatform = await ArticleHelper.numberOfSharesOnPlatform( |
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
-user can get ratings, likes AND dislikes on an article while not authenticated. [delivers #167703750]
63b2e07
to
698a54a
Compare
Description
a user should be able to see the ratings, likes, and dislikes on an article even when they are not authenticated.
How has it been tested
npm test
Checklist:
N/A
Pivotal tracker story ID
#167703750