Implement sources API routes #73

Merged

caradoc merged 1 commit from feat/task-fa0bdcd5-sources-api into main

2026-05-30 15:37:54 -05:00

caradoc commented

2026-05-30 14:56:26 -05:00

Collaborator

Summary

add source persistence and source/contact associations
add authenticated sources API CRUD routes
document sources routes in OpenAPI and add route/migration tests

Verification

make check
./scripts/pre-pr.sh

Task: #task-fa0bdcd5

## Summary - add source persistence and source/contact associations - add authenticated sources API CRUD routes - document sources routes in OpenAPI and add route/migration tests ## Verification - `make check` - `./scripts/pre-pr.sh` Task: #task-fa0bdcd5

caradoc added 1 commit

2026-05-30 14:56:26 -05:00

feat: implement sources API routes

CI / build-lint-test (pull_request) Successful in 1m28s

Details

4b53c6a229

Task: #task-fa0bdcd5

caradoc force-pushed feat/task-fa0bdcd5-sources-api from 4b53c6a229

CI / build-lint-test (pull_request) Successful in 1m28s

Details

to 91c13245ac

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

Details

2026-05-30 14:58:44 -05:00

Compare

caradoc commented

2026-05-30 14:59:32 -05:00

Author

Collaborator

PR review approved

Summary:

Adds sources persistence and source/contact association storage.
Adds authenticated sources CRUD routes with shared validation, not-found, and conflict responses.
Documents sources routes and schemas in OpenAPI.
Adds route and migration test coverage.

Acceptance criteria:

Can list sources.
Can create a source with a URL.
Can create a source without a URL.
Can get, update, and delete a source by ID.
Can associate source contacts.
Invalid optional URLs and missing contact IDs return shared validation errors.
Feed URLs are represented as source-owned accounts, not feedUrl fields.
Every sources route appears in OpenAPI with documented operation IDs.
Relevant lint/test checks pass.

Blocking issues: none.

Warnings: none.

Verification:

make check passed.
./scripts/pre-pr.sh passed.
Forgejo CI CI / build-lint-test (pull_request) passed.

Final verdict: approved.

### PR review approved Summary: - Adds sources persistence and source/contact association storage. - Adds authenticated sources CRUD routes with shared validation, not-found, and conflict responses. - Documents sources routes and schemas in OpenAPI. - Adds route and migration test coverage. Acceptance criteria: - [x] Can list sources. - [x] Can create a source with a URL. - [x] Can create a source without a URL. - [x] Can get, update, and delete a source by ID. - [x] Can associate source contacts. - [x] Invalid optional URLs and missing contact IDs return shared validation errors. - [x] Feed URLs are represented as source-owned accounts, not `feedUrl` fields. - [x] Every sources route appears in OpenAPI with documented operation IDs. - [x] Relevant lint/test checks pass. Blocking issues: none. Warnings: none. Verification: - `make check` passed. - `./scripts/pre-pr.sh` passed. - Forgejo CI `CI / build-lint-test (pull_request)` passed. Final verdict: approved.

caradoc referenced this pull request

2026-05-30 15:00:21 -05:00

Implement sources API routes #26

caradoc merged commit 7cf77c7a57 into main

2026-05-30 15:37:54 -05:00

caradoc deleted branch feat/task-fa0bdcd5-sources-api

2026-05-30 15:37:54 -05:00

caradoc referenced this pull request from a commit

2026-05-30 15:37:56 -05:00

Merge pull request 'Implement sources API routes' (#73) from feat/task-fa0bdcd5-sources-api into main

caradoc referenced this pull request

2026-05-30 15:40:43 -05:00