Skip to content

Mail reader shows the sender check and reply publication (#138) - #35

Merged
theobong merged 1 commit into
mainfrom
feat/mail-reply-verdict-badges
Sep 23, 2026
Merged

theobong merged 1 commit into
mainfrom
feat/mail-reply-verdict-badges

Conversation

@theobong

Copy link
Copy Markdown
Member

Part of civfix/issue-tracker#138

What changed

The admin mail reader now shows whether a city reply passed the sender check and whether it was published to the report or event, and the Inbox reader shows the sender check on each email. Admin.

Before you start

  • Where: staging (admin.civfix.dev) after the main push
  • Sign in as: operator
  • Data: a mail thread linked to a report or event that holds a city reply, and an inbox email; staging shows only the replies and emails it already holds

Verify

[Admin]

  1. On the Dashboard, click the "Mail" tile header — Expect: the Mail page opens on "Outreach".
  2. Click the "Inbound" chip and open a thread that shows "View report" and has a city reply — Expect: next to the city message's time is a sender-check pill: green "Verified sender", red "Failed sender check" or amber "Sender not checked"; hovering "Verified sender" shows "The sender's domain passed authentication."
  3. Look at the second pill on that message — Expect: "Published", "Publishing" or "Withheld"; hovering "Published" shows "Posted to the report chat and timeline."
  4. Look at the messages sent from CivFix in the same thread — Expect: no sender-check or publication pill.
  5. Open a thread that shows "View jurisdiction" but not "View report" and has a city reply — Expect: a sender-check pill but no publication pill.
  6. Click "Inbox" and open an email — Expect: a sender-check pill follows "to " in the reader header when the email carries one.

Regression

Outreach status pills — [Admin]

  1. Click "Outreach" and look at the row pills and the reader header pill — Expect: "Sent", "Delivered", "Opened", "Replied", "Auto-reply", "Needs action" and "Bounced" keep their labels and colors.

Replying on a thread — [Admin]

  1. Type in the "Reply to …" box and click "Reply" — Expect: toast "Reply sent to "; the new message has no sender-check pill.
  2. Click "Mark replied" — Expect: toast "Marked replied"; the header pill reads "Replied".

Handling an inbox email — [Admin]

  1. Click "Inbox", open an unread email and click "Mark read" — Expect: toast "Marked read".
  2. Click "Archive" — Expect: toast "Archived".

Opening a thread from a report — [Admin]

  1. Open a report that has outreach and click "View conversation →" — Expect: Mail opens on "Outreach" with that thread and its pills.

Not covered

  • Mail stored before sender checks were recorded shows no sender-check pill, by design.

@theobong
theobong merged commit f2bccd9 into main Sep 23, 2026
1 check passed
@theobong
theobong deleted the feat/mail-reply-verdict-badges branch September 23, 2026 22:24
@greptile-apps

greptile-apps Bot commented Sep 23, 2026

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

Safe to merge.

What we checked:

  • Rendered the production badge components for all supported sender-authentication verdicts and reply-publication states, and verified the expected labels, semantic status classes, and tooltips, with before-and-after browser recordings showing the reader metadata before the change and after the badges were applied. T-Rex
  • Ran unit tests and confirmed all 3 rendered-component checks passed in Vitest. T-Rex
  • Used Chromium Playwright to capture before/after WebM evidence and verified that every expected label, semantic status class, and tooltip appeared as intended. T-Rex

Reviews (1) · Last reviewed commit: "mail reader shows sender verification an..."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant