Skip to content

Render Human answer activity - #650

Merged
willmot merged 7 commits into
mainfrom
codex/human-mentions
Sep 2, 2026
Merged

Render Human answer activity#650
willmot merged 7 commits into
mainfrom
codex/human-mentions

Conversation

@willmot

@willmot willmot commented Aug 25, 2026

Copy link
Copy Markdown
Member
  • render human_mention comments as quiet activity rows in the H2 comment stream
  • show “Referenced by @human to answer a question from @user”, linking the question to Slack and the user to their H2 profile
  • share the established activity-row layout with Slack mention markers
  • keep activity markers out of comment counts, avatar piles, and Recent Comments
  • cover both activity rows in the existing Storybook comments story

Human Bot producer: https://github.com/humanmade/human-bot/pull/210

Visual

Screenshot 2026-08-25 at 18 09 32

Show email-silent human_mention comments as quiet activity rows beside Slack mention markers, using H2's existing Human Made logo and excluding markers from comment counts.

Co-authored-by: Codex <noreply@openai.com>
@github-actions

github-actions Bot commented Aug 25, 2026

Copy link
Copy Markdown

🛝 Playground preview

Open PR #650 in WordPress Playground

Spins up a fresh WordPress with this PR's build of the H2 theme and demo content, logged in as admin.

Built from fc5aca5 — the preview updates on every push to this PR.

Seed a Human answer marker and install the minimal demo-only REST read filter that hmn.md provides in production, so local and PR Playground previews exercise the real activity row.

Co-authored-by: Codex <noreply@openai.com>
github-actions Bot added a commit that referenced this pull request Aug 25, 2026
github-actions Bot added a commit that referenced this pull request Aug 25, 2026
@rmccue

rmccue commented Aug 25, 2026

Copy link
Copy Markdown
Member

Looks like a rendering issue with the avatar's alignment:

Screenshot 2026-08-25 at 16 01 48@2x

Might not be from this PR - the prior changes for migrating to Tailwind I don't think had incorporated the mini update comments.

github-actions Bot added a commit that referenced this pull request Aug 25, 2026
github-actions Bot added a commit that referenced this pull request Aug 25, 2026
Co-authored-by: OpenAI Codex <codex@openai.com>
github-actions Bot added a commit that referenced this pull request Aug 25, 2026
@willmot
willmot marked this pull request as ready for review August 25, 2026 16:04
@willmot

willmot commented Aug 25, 2026

Copy link
Copy Markdown
Member Author

@rmccue ready for review/merge. Not sure if you actually want the Playground additions, can remove those if not.

@Human

Human commented Aug 25, 2026

Copy link
Copy Markdown

show “Referenced by @Human to answer a question from @user”, linking the question to Slack and the user to their H2 profile

I do not need to be notified, thanks.

@willmot
willmot requested a review from rmccue August 26, 2026 14:01
Remove the PR-specific Playground shim and seeded markers. Keep the visual examples in H2's established component development surface instead.

Co-authored-by: OpenAI Codex <codex@openai.com>
github-actions Bot added a commit that referenced this pull request Sep 2, 2026

@rmccue rmccue left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved - that said I think it might be worth a follow-up PR separately to make this a little more generic, as the use of "human" eg is tied directly to the name of our bot, not to any bot that a H2 user might have (and also meant the PR accidentally pinged above - sorry about that!)

@rmccue

rmccue commented Sep 2, 2026

Copy link
Copy Markdown
Member

Also: I would update the Playground preview here, I think it's liable to get lost if it's just in Storybook, and it's handy for people previewing H2 to see what that looks like.

@willmot
willmot merged commit 68dce67 into main Sep 2, 2026
3 of 4 checks passed
@willmot
willmot deleted the codex/human-mentions branch September 2, 2026 17:21
@willmot

willmot commented Sep 2, 2026

Copy link
Copy Markdown
Member Author

Follow-up opened in #653 to make the activity renderer generic while preserving existing Human markers.

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.

3 participants