Implement comments API routes #70

Merged

caradoc merged 1 commit from feat/task-ac6a35a8-comments-api-routes into main

2026-05-24 10:00:09 -05:00

caradoc commented

2026-05-24 09:08:45 -05:00

Collaborator

Summary

Add comments persistence and moderation helpers for post replies.
Add authenticated comments API routes for post comment listing, pending moderation, approve, and hide actions.
Document comments routes in OpenAPI and add route/migration tests.

Verification

make check
./scripts/pre-pr.sh

Task: #task-ac6a35a8

## Summary - Add comments persistence and moderation helpers for post replies. - Add authenticated comments API routes for post comment listing, pending moderation, approve, and hide actions. - Document comments routes in OpenAPI and add route/migration tests. ## Verification - make check - ./scripts/pre-pr.sh Task: #task-ac6a35a8

caradoc added 1 commit

2026-05-24 09:08:45 -05:00

feat: implement comments API routes

CI / build-lint-test (pull_request) Successful in 31s

Details

70eaa59944

Task: #task-ac6a35a8

caradoc commented

2026-05-24 09:10:03 -05:00

Author

Collaborator

PR Review: Approved

Summary

The PR implements the comments API slice cleanly: persistence, authenticated route handlers, OpenAPI documentation, and coverage for the expected moderation/listing flows are all present. Local verification and Forgejo CI are green.

Acceptance Criteria

Can list comments for a post.
Can list pending comments across posts.
Can approve a comment.
Can hide a comment.
Missing post/comment returns shared 404.
Unsupported implementations return standardized 501 Not Implemented.
Every route appears in OpenAPI with documented operation IDs.
Relevant lint/test checks pass.

Blocking Issues

None.

Warnings

None.

Final Verdict

Approved. Stop at the human merge gate.

## PR Review: Approved ### Summary The PR implements the comments API slice cleanly: persistence, authenticated route handlers, OpenAPI documentation, and coverage for the expected moderation/listing flows are all present. Local verification and Forgejo CI are green. ### Acceptance Criteria - [x] Can list comments for a post. - [x] Can list pending comments across posts. - [x] Can approve a comment. - [x] Can hide a comment. - [x] Missing post/comment returns shared `404`. - [x] Unsupported implementations return standardized `501 Not Implemented`. - [x] Every route appears in OpenAPI with documented operation IDs. - [x] Relevant lint/test checks pass. ### Blocking Issues None. ### Warnings None. ### Final Verdict Approved. Stop at the human merge gate.

caradoc referenced this pull request

2026-05-24 09:14:28 -05:00

Implement comments API routes #18

caradoc referenced this pull request

2026-05-24 09:14:28 -05:00

Implement comments API routes #18

caradoc merged commit 918a6d1d24 into main

2026-05-24 10:00:09 -05:00

caradoc deleted branch feat/task-ac6a35a8-comments-api-routes

2026-05-24 10:00:09 -05:00

caradoc referenced this pull request from a commit

2026-05-24 10:00:10 -05:00