-
Notifications
You must be signed in to change notification settings - Fork 1
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
#162727487 comment update history #30
#162727487 comment update history #30
Conversation
[Feature #162727487]
…into feature/162727487/comment-update-history Merge staging to current branch .
Write test for edit history implement get all edit histroy implement create edit history write validation for comment update body [Feature #162727487]
@@ -0,0 +1,23 @@ | |||
module.exports = { |
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.
'module' is not defined no-undef
const { body } = req.body; | ||
const { commentId } = req.params; | ||
const { id } = req.verifyUser; | ||
const findComment = await Comment.find({ |
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.
Can we move this to the middleware. I think it will make more sense there.
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.
You didn't fix this....
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.
Oh just seeing this. will do now
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.
Done
[Fixed #162727487/]
@@ -0,0 +1,58 @@ | |||
import models from '../db/models'; |
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.
Parsing error: 'import' and 'export' may appear only with 'sourceType: module'
…into feature/162727487/comment-update-history
[Fixed #162727487]
What does this PR do?
This PR implements comment edit history track
Description of Task to be completed?
Comment edit history can be saved and be retrieved
How should this be manually tested?
On the update comment route PUT
/api/v1/arts/comments/commentId
Update a comment with a certain more as many times as you want
Hit GET
/api/v1/arts/coments/commentId
with same ID and get all the edit historyAny background context you want to provide?
Run
npm run db:seed
before testingWhat are the relevant pivotal tracker stories?
#162727487
Screenshots (if appropriate)
None
Questions:
None