Add incoming/outgoing direction to communications - #2177
Merged
Conversation
maebeale
force-pushed
the
maebeale/comms-incoming-designation
branch
from
August 13, 2026 13:52
0b5b8ec to
3f7de1b
Compare
- Record whether a communication is incoming (sent by the person it's about) via a direction slider on the log forms; default outgoing. - Flag exceptions with chips on the index/detail/box: sky Incoming, grey FYI, indigo Bulk (bulk_payment); resolve recipient emails to person names with the email on hover in To/From. - Theme the communications index and detail with the notifications colour, add a back eyebrow to the originating record, grey top eyebrows, and clickable comms-box rows (new tab). - Tint the inline comment/communication log forms with their domain colours. - Misc: rename the registration ticket's "Edit registration" chip. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
maebeale
force-pushed
the
maebeale/comms-incoming-designation
branch
from
August 13, 2026 13:57
3f7de1b to
73655c6
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🤖 suggested review level: 5 Inspect 🔬 new column + backfill-safe migration, decorator/policy-adjacent logic, and wide comms UI changes
Goal
What changed
directionstring column (outgoingdefault, reversible migration) with an "Incoming" slider on the log forms; From/To flip for incoming.Notes
row_class/card_bg_class/archived?notification styling.bulk_payment_*kinds; bulk reminders/invites carry no per-notification marker, so they aren't flagged (call out if you want a marker added).