Skip to content

Conversation

@SandipBajracharya
Copy link
Collaborator

Changes

  • accept comment id as path variable
  • include attachments in response
  • serialize response data

Testing Criteria

Loom

@linear
Copy link

linear bot commented Jan 15, 2026

@vercel
Copy link
Contributor

vercel bot commented Jan 15, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
tasks-app Ready Ready Preview, Comment Jan 21, 2026 8:05am

Request Review

Copy link
Collaborator

@arpandhakal arpandhakal left a comment

Choose a reason for hiding this comment

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

src/app/api/comment/public/comment-public.controller.ts

Lets follow naming conventions here. Since we are already on the comment/public route the name of the file should be just public.controller.ts.

@arpandhakal arpandhakal self-requested a review January 20, 2026 09:24
@SandipBajracharya SandipBajracharya merged commit 899c6d8 into feature/api-improvements Jan 21, 2026
3 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.

3 participants