-
Notifications
You must be signed in to change notification settings - Fork 5
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
164796910 Adds Comment Edit History feature #80
Conversation
a577d64
to
a916e2b
Compare
@@ -83,7 +83,7 @@ describe('Util test', () => { | |||
}); | |||
}); | |||
|
|||
it('should return error 2', () => { | |||
it('should return error', () => { |
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.
Replace 'should·return·error'
with "should·return·error"
prettier/prettier
Strings must use doublequote quotes
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.
This has been fixed a long time ago. It should be a problem with HoundCI server
const res = {}; | ||
const next = sinon.spy(); | ||
trimBody(req, res, next); | ||
req.body.should.deep.equal({ name: 'Micah', seenGot: true }); |
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.
Replace 'Micah'
with "Micah"
prettier/prettier
Strings must use doublequote quotes
|
||
it('should trim request body', () => { | ||
const req = { | ||
body: { name: ' Micah ', seenGot: true } |
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.
Replace '···Micah·'
with "···Micah·"
prettier/prettier
Strings must use doublequote quotes
next.called.should.equal(true); | ||
}); | ||
|
||
it('should trim request body', () => { |
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.
Replace 'should·trim·request·body'
with "should·trim·request·body"
prettier/prettier
Strings must use doublequote quotes
const res = {}; | ||
const next = sinon.spy(); | ||
trimBody(req, res, next); | ||
req.body.should.deep.equal({ name: 'Micah', cohort: '46' }); |
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.
Replace 'Micah',·cohort:·'46'
with "Micah",·cohort:·"46"
prettier/prettier
Strings must use doublequote quotes
@@ -0,0 +1,29 @@ | |||
import 'chai/register-should'; |
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.
Definition for rule 'jsx-a11y/anchor-is-valid' was not found jsx-a11y/anchor-is-valid
Replace 'chai/register-should'
with "chai/register-should"
prettier/prettier
Strings must use doublequote quotes
tests/mock/comment.js
Outdated
invalidarticleUUID, | ||
validCommentId: '979eaa2e-5b8f-4103-8192-4639afae2bb9', | ||
validArticleId: '979eaa2e-5b8f-4103-8192-4639afae2ba8', | ||
invalidCommentId: '979eaa2e-5b8f-4103-8192-4639afae2bb' |
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.
Replace '979eaa2e-5b8f-4103-8192-4639afae2bb'
with "979eaa2e-5b8f-4103-8192-4639afae2bb"
prettier/prettier
Strings must use doublequote quotes
tests/mock/comment.js
Outdated
invalidComment, | ||
invalidarticleUUID, | ||
validCommentId: '979eaa2e-5b8f-4103-8192-4639afae2bb9', | ||
validArticleId: '979eaa2e-5b8f-4103-8192-4639afae2ba8', |
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.
Replace '979eaa2e-5b8f-4103-8192-4639afae2ba8'
with "979eaa2e-5b8f-4103-8192-4639afae2ba8"
prettier/prettier
Strings must use doublequote quotes
tests/mock/comment.js
Outdated
uncompleteCommentBody, | ||
invalidComment, | ||
invalidarticleUUID, | ||
validCommentId: '979eaa2e-5b8f-4103-8192-4639afae2bb9', |
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.
Replace '979eaa2e-5b8f-4103-8192-4639afae2bb9'
with "979eaa2e-5b8f-4103-8192-4639afae2bb9"
prettier/prettier
Strings must use doublequote quotes
if (err) return done(err); | ||
const { status, body } = res; | ||
status.should.equal(500); | ||
body.status.should.equal('error'); |
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.
Replace 'error'
with "error"
prettier/prettier
Strings must use doublequote quotes
8fac461
to
8a34522
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
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.
Good implementation @micah-akpan
LGTM
LGTM |
f552dd5
to
730e9b5
Compare
Change function names and make other minor code changes [#164796892]
Remove console statement [#164796910]
730e9b5
to
742b5ca
Compare
Description
Add a comment edit history feature to facilitate the retrieval of a comment's edit history
Type of change
New feature (non-breaking change that adds functionality)
How Has This Been Tested?
Checklist:
Screenshots
(no edits yet)
(new edits has been made)
PT
164796910