feat(feedback-site): a name-rating page at /names (#1257) #1264

Merged
claude-bot merged 2 commits from feat/1257-names-feedback into main 2026-10-28 16:07:31 +00:00
Collaborator

Part of #1257. This lets the owner's testers, who have no claude.ai accounts, draw names from the built-in generator (no AI), mark each 👍/👎 and leave notes per style.

Already deployed to the dev feedback site (docker-test ~/projects/qb-feedback), with the responses backed up first (qb-feedback-responses-20261028T151149Z.tgz). The page is at https://questboard-dev.rhoving.com/feedback/names. The owner's 22 sampler marks are saved as one rater.

  • static/names.html: the generator is copied byte-for-byte from the sampler. Opening the page and drawing names needs no key; saving needs the share key, as on the other pages. It autosaves.
  • app.py: a names audience with its own strict NamesDoc (allowed styles, up/down votes, one vote per name, size limits) and its own PUT /api/responses/names/{rid}, registered ahead of the generic route. Round one and two behave as before (shared _write_record). Names are left out of the CSV export, whose columns are unchanged.
  • admin.html: a Names section with up/down counts per name and every rater's notes. The round-one filter no longer picks up names documents.
  • Tests: tests/test_names.py. The suite has 152 passed (run in a venv).
  • README section and a changelog fragment.

Not checked in a browser at phone width; the page is live on dev to look at.

🤖 Generated with Claude Code

Part of #1257. This lets the owner's testers, who have no claude.ai accounts, draw names from the built-in generator (no AI), mark each 👍/👎 and leave notes per style. **Already deployed** to the dev feedback site (docker-test `~/projects/qb-feedback`), with the responses backed up first (`qb-feedback-responses-20261028T151149Z.tgz`). The page is at https://questboard-dev.rhoving.com/feedback/names. The owner's 22 sampler marks are saved as one rater. - `static/names.html`: the generator is copied byte-for-byte from the sampler. Opening the page and drawing names needs no key; saving needs the share key, as on the other pages. It autosaves. - `app.py`: a `names` audience with its own strict `NamesDoc` (allowed styles, up/down votes, one vote per name, size limits) and its own `PUT /api/responses/names/{rid}`, registered ahead of the generic route. Round one and two behave as before (shared `_write_record`). Names are left out of the CSV export, whose columns are unchanged. - `admin.html`: a Names section with up/down counts per name and every rater's notes. The round-one filter no longer picks up names documents. - Tests: `tests/test_names.py`. The suite has 152 passed (run in a venv). - README section and a changelog fragment. Not checked in a browser at phone width; the page is live on dev to look at. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
feat(feedback-site): names rating page (#1257)
Some checks failed
CI / Does this change touch the recording path? (pull_request) Successful in 17s
CI / Synthetic session harness (no GPU, no LLM) (pull_request) Has been skipped
CI / Which journeys does this change need? (pull_request) Successful in 18s
CI / Bot/backend version sync (pull_request) Successful in 1m4s
CI / Backend lint (ruff) (pull_request) Successful in 1m17s
CI / Feedback site tests (pull_request) Successful in 1m50s
CI / Summarisation accuracy eval harness (stub provider) (pull_request) Successful in 2m0s
CI / Bot tests and audit (pull_request) Successful in 3m25s
CI / Docker image build (pull_request) Successful in 4m51s
CI / End-to-end journeys (desktop) (pull_request) Has been skipped
CI / Frontend (shard 2/2) (pull_request) Successful in 7m47s
CI / Frontend (shard 1/2) (pull_request) Successful in 9m15s
CI / Backend migration, tests, and audit (pull_request) Has been cancelled
aef0579a73
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
claude-bot scheduled this pull request to auto merge when all checks succeed 2026-10-28 15:26:55 +00:00
feedback-site: plainer names page intro, steps as a list (#1257)
All checks were successful
CI / Does this change touch the recording path? (pull_request) Successful in 16s
CI / Which journeys does this change need? (pull_request) Successful in 17s
CI / Synthetic session harness (no GPU, no LLM) (pull_request) Has been skipped
CI / Summarisation accuracy eval harness (stub provider) (pull_request) Successful in 59s
CI / Backend lint (ruff) (pull_request) Successful in 1m3s
CI / Bot/backend version sync (pull_request) Successful in 1m3s
CI / Feedback site tests (pull_request) Successful in 1m49s
CI / Bot tests and audit (pull_request) Successful in 3m25s
CI / Docker image build (pull_request) Successful in 5m52s
CI / End-to-end journeys (desktop) (pull_request) Has been skipped
CI / Frontend (shard 2/2) (pull_request) Successful in 8m23s
CI / Frontend (shard 1/2) (pull_request) Successful in 9m46s
CI / Backend migration, tests, and audit (pull_request) Successful in 33m52s
7e66123619
One background sentence, a numbered How to review, and the save state as a
plain line that turns into a warning box only when ratings can't be saved.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
claude-bot deleted branch feat/1257-names-feedback 2026-10-28 16:07:31 +00:00
Sign in to join this conversation.
No description provided.