fix(admin-feedback): show name + masked email on each row + toggle Reviewed #102

Merged

YousifShkara merged 1 commit from fix/feedback-admin-row into main

2026-06-10 08:47:56 +02:00

YousifShkara commented

2026-06-10 08:47:53 +02:00

Owner

Two small fixes to the admin /feedback triage list.

Show the submitter on each row. AdminFeedbackSummary from bunyip-api already carries name: Option<String> and email_masked: Option<String>; both were dropped on the floor in the row template. They now render as a Name · email@masked.com line under the subject, with the obvious None-handling: only the name when no email was given, only the email when no name was given, the whole line suppressed when both are missing. No DTO change.

Make "Reviewed" toggle. With a row already showing the "reviewed" badge and the unchanged "Reviewed" button right next to it, clicking did a no-op POST that re-set the same status. The button now flips behaviour off the current status: on a row whose status == Reviewed it labels "Un-review" and posts status=new, sending the row back to the triage queue. On every other status it stays "Reviewed" / status=reviewed. Closed rows still expose the Reviewed button so a row can be re-opened with closed -> reviewed -> new without touching the API directly.

The Close button is unchanged.

Two small fixes to the admin /feedback triage list. Show the submitter on each row. AdminFeedbackSummary from bunyip-api already carries `name: Option<String>` and `email_masked: Option<String>`; both were dropped on the floor in the row template. They now render as a `Name · email@masked.com` line under the subject, with the obvious None-handling: only the name when no email was given, only the email when no name was given, the whole line suppressed when both are missing. No DTO change. Make "Reviewed" toggle. With a row already showing the "reviewed" badge and the unchanged "Reviewed" button right next to it, clicking did a no-op POST that re-set the same status. The button now flips behaviour off the current status: on a row whose `status == Reviewed` it labels "Un-review" and posts `status=new`, sending the row back to the triage queue. On every other status it stays "Reviewed" / `status=reviewed`. Closed rows still expose the Reviewed button so a row can be re-opened with closed -> reviewed -> new without touching the API directly. The Close button is unchanged.

YousifShkara self-assigned this

2026-06-10 08:47:53 +02:00

YousifShkara added 1 commit

2026-06-10 08:47:53 +02:00

fix(admin-feedback): show name + masked email on each row + toggle Reviewed

Create release / Create release from merged PR (pull_request) Has been skipped

Details

Check / fmt / clippy / build / test (pull_request) Successful in 1m30s

Details

70191f4803

Two small fixes to the admin /feedback triage list.

Show the submitter on each row. AdminFeedbackSummary from bunyip-api already carries `name: Option<String>` and `email_masked: Option<String>`; both were dropped on the floor in the row template. They now render as a `Name · email@masked.com` line under the subject, with the obvious None-handling: only the name when no email was given, only the email when no name was given, the whole line suppressed when both are missing. No DTO change.

Make "Reviewed" toggle. With a row already showing the "reviewed" badge and the unchanged "Reviewed" button right next to it, clicking did a no-op POST that re-set the same status. The button now flips behaviour off the current status: on a row whose `status == Reviewed` it labels "Un-review" and posts `status=new`, sending the row back to the triage queue. On every other status it stays "Reviewed" / `status=reviewed`. Closed rows still expose the Reviewed button so a row can be re-opened with closed -> reviewed -> new without touching the API directly.

The Close button is unchanged.

YousifShkara merged commit 630e1db08d into main

2026-06-10 08:47:56 +02:00

YousifShkara deleted branch fix/feedback-admin-row

2026-06-10 08:47:56 +02:00

YousifShkara referenced this pull request from a commit

2026-06-10 08:47:58 +02:00

Merge pull request 'fix(admin-feedback): show name + masked email on each row + toggle Reviewed' (#102) from fix/feedback-admin-row into main