Skip to content

Conversation

pshroff04
Copy link
Collaborator

Change log description
Added code for posting (single) comment.

Purpose of the change
Partially fixes #8

What the code does

The following request has been tested locally to return 200 OK null response and verified the entry in DB.
http://localhost:8080/comment/postComment

How to verify it
CI Build shall pass.
Testing via API call : curl -H "Content-Type: application/json" -X POST --data '{"comment":"This is test comment","parentCommentId":"","userName": "prateek","contentId":"c160d3e029104224aab7db09f91633c1"}' http://localhost:8080/comment/postComment

@pshroff04 pshroff04 requested a review from Tristan1900 January 3, 2021 10:38
@pshroff04
Copy link
Collaborator Author

@vaybhavshaw @ManishKumarKeshri Take a look.

Copy link
Collaborator

@ManishKumarKeshri ManishKumarKeshri left a comment

Choose a reason for hiding this comment

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

Left some comments.

@ManishKumarKeshri ManishKumarKeshri requested review from vaybhavshaw and Tristan1900 and removed request for Tristan1900 and vaybhavshaw January 6, 2021 04:03
@pshroff04 pshroff04 changed the title Initial code for posting comments. Added logging too. Added: Comment-Service, Logging, Mongo persistence API logic. Jan 17, 2021
@pshroff04
Copy link
Collaborator Author

Done.

* @return A Response entity which will have all the user details.
*/
@RequestMapping(method = RequestMethod.POST, value = "/postComment", produces = MediaType.APPLICATION_JSON_VALUE)
public ResponseEntity<?> postComment(@RequestBody String commentJson) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Will it be immediately reflected?

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think No. But we will have to look into that logic.

@ManishKumarKeshri ManishKumarKeshri merged commit 50a8ae9 into TechDiscussion:master Jan 24, 2021
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.

Updation
3 participants