Implement accounts API routes #74

Merged

caradoc merged 1 commit from feat/task-42da73dd-accounts-api into main

2026-05-30 16:49:38 -05:00

caradoc commented

2026-05-30 16:38:39 -05:00

Collaborator

Summary

add account persistence for contact, source, and site owners
add authenticated accounts API CRUD routes with owner filtering
embed source-owned accounts in source responses
document account routes in OpenAPI and add route/migration tests

Verification

make check
./scripts/pre-pr.sh

Task: #task-42da73dd

## Summary - add account persistence for contact, source, and site owners - add authenticated accounts API CRUD routes with owner filtering - embed source-owned accounts in source responses - document account routes in OpenAPI and add route/migration tests ## Verification - `make check` - `./scripts/pre-pr.sh` Task: #task-42da73dd

caradoc added 1 commit

2026-05-30 16:38:39 -05:00

feat: implement accounts API routes

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

Details

a8a8e0859e

Task: #task-42da73dd

caradoc commented

2026-05-30 16:39:19 -05:00

Author

Collaborator

PR review approved

Summary:

Adds account persistence for contact, source, and site owners.
Adds authenticated accounts CRUD routes with owner filtering and shared validation/not-found responses.
Enforces deterministic default-account behavior by clearing other defaults for the same owner.
Embeds source-owned accounts in source responses.
Adds OpenAPI route/schema coverage and route/migration tests.

Acceptance criteria:

Can list accounts.
Can create contact-owned, source-owned, and site-owned accounts.
Can get, update, and delete an account by ID.
Invalid owners return shared validation/not-found errors.
Default account behavior is deterministic and tested.
Source feed URLs can be represented with ownerType: "source", kind: "feed", and protocol: "rss".
Every accounts 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 account persistence for contact, source, and site owners. - Adds authenticated accounts CRUD routes with owner filtering and shared validation/not-found responses. - Enforces deterministic default-account behavior by clearing other defaults for the same owner. - Embeds source-owned accounts in source responses. - Adds OpenAPI route/schema coverage and route/migration tests. Acceptance criteria: - [x] Can list accounts. - [x] Can create contact-owned, source-owned, and site-owned accounts. - [x] Can get, update, and delete an account by ID. - [x] Invalid owners return shared validation/not-found errors. - [x] Default account behavior is deterministic and tested. - [x] Source feed URLs can be represented with `ownerType: "source"`, `kind: "feed"`, and `protocol: "rss"`. - [x] Every accounts 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 16:41:23 -05:00

Implement accounts API routes #25

caradoc merged commit 44858aa1cc into main

2026-05-30 16:49:38 -05:00

caradoc deleted branch feat/task-42da73dd-accounts-api

2026-05-30 16:49:38 -05:00

caradoc referenced this pull request from a commit

2026-05-30 16:49:38 -05:00

Merge pull request 'Implement accounts API routes' (#74) from feat/task-42da73dd-accounts-api into main

caradoc referenced this pull request

2026-05-30 18:37:39 -05:00