Skip to content

feat/264: create endpoint and handler for GetCommentById#276

Merged
ArturOleksiuk merged 11 commits into
devfrom
feat/264/comments_get_by_id_endpoint
Dec 30, 2025
Merged

feat/264: create endpoint and handler for GetCommentById#276
ArturOleksiuk merged 11 commits into
devfrom
feat/264/comments_get_by_id_endpoint

Conversation

@ArturOleksiuk

@ArturOleksiuk ArturOleksiuk commented Dec 29, 2025

Copy link
Copy Markdown
Contributor

dev

GitHub

Code reviewers

Summary of issue

Implement GetCommentById endpoint with MediatR handler and specification to retrieve a comment with its replies.

CHECK LIST

  • СI passed
  • Сode coverage >=95%
  • PR is reviewed manually again (to make sure you have 100% ready code)
  • All reviewers agreed to merge the PR
  • I've checked new feature as logged in and logged out user if needed
  • PR meets all conventions

@ArturOleksiuk ArturOleksiuk force-pushed the feat/264/comments_get_by_id_endpoint branch from ceba5c0 to a7149c7 Compare December 30, 2025 00:39
@ArturOleksiuk ArturOleksiuk force-pushed the feat/264/comments_get_by_id_endpoint branch from 7c578b5 to c7a907c Compare December 30, 2025 04:24

@Ammoniy Ammoniy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Separate fature an tests

@ArturOleksiuk

Copy link
Copy Markdown
Contributor Author

Separate fature an tests

I wasn’t able to work on the separate feature and tests yet because the main feature logic hasn’t been approved. I added the tests to the main feature for now.

@LekhivOleh LekhivOleh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm 👍

@sonarqubecloud

Copy link
Copy Markdown

@ArturOleksiuk ArturOleksiuk merged commit acdf11f into dev Dec 30, 2025
5 checks passed
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.

[Task] GetById Comment With Replies to Review

3 participants