Skip to content

Add admin visibility and affiliation indicators on people index#1273

Merged
maebeale merged 3 commits intomainfrom
maebeale/people-published-scope
Mar 1, 2026
Merged

Add admin visibility and affiliation indicators on people index#1273
maebeale merged 3 commits intomainfrom
maebeale/people-published-scope

Conversation

@maebeale
Copy link
Collaborator

@maebeale maebeale commented Mar 1, 2026

What is the goal of this PR and why is this important?

  • Admins need to quickly identify unpublished people in the people index
  • Affiliation labels should distinguish facilitator vs non-facilitator roles
  • Profile button hover was inflating Ahoy view counts via Turbo prefetch

How did you approach the change?

  • Added admin-only bg-blue-100 row styling for unpublished people, matching the existing workshop index pattern
  • Changed affiliation column to iterate over affiliations (not organizations) so per-affiliation styling can be applied
  • Added grey left border on non-facilitator affiliation labels to visually distinguish facilitator affiliations
  • Added turbo_prefetch: false to person_profile_button and organization_profile_button helpers to prevent hover prefetch from triggering Ahoy view tracking

UI Testing Checklist

  • People index shows blue background on rows for unpublished people (not searchable or no active affiliations)
  • Facilitator affiliation labels have no left border; non-facilitator labels have grey left border
  • Hovering profile buttons does NOT trigger network requests to show pages
  • Clicking profile buttons still navigates correctly

Anything else to add?

  • The turbo_prefetch: false fix applies globally to all callers of both helpers, not just the people index

🤖 Generated with Claude Code

maebeale and others added 3 commits March 1, 2026 05:35
…indicators on people index

- Highlight unpublished people rows with admin-only blue background
- Iterate over affiliations instead of organizations for per-affiliation styling
- Add grey left border on non-facilitator affiliation labels to visually
  distinguish facilitator affiliations

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Hovering these links triggered Turbo prefetch, which loaded the show
page and inflated Ahoy view counts. This adds turbo_prefetch: false
to both helpers, matching the pattern used on workshop, resource, and
story links.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@maebeale maebeale merged commit 6d8e37f into main Mar 1, 2026
3 checks passed
@maebeale maebeale deleted the maebeale/people-published-scope branch March 1, 2026 10:56
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