Skip to content

feat(team): click-tracking on Report wrong + member-profile deep link#3469

Merged
bokelley merged 1 commit into
mainfrom
bokelley/team-page-polish-and-deep-link
Apr 28, 2026
Merged

feat(team): click-tracking on Report wrong + member-profile deep link#3469
bokelley merged 1 commit into
mainfrom
bokelley/team-page-polish-and-deep-link

Conversation

@bokelley

Copy link
Copy Markdown
Contributor

Summary

Two polish items deferred from the PR #3450 review (product expert recommendations):

  1. Click-tracking on "Report wrong" — new endpoint POST /api/organizations/:orgId/brand-classification-report records {orgId, userId, kind, subject_domain} to registry_audit_log on every click. The frontend fires navigator.sendBeacon from the anchor's click handler; the existing mailto: opens normally. This gives us a triage queue and lets us detect when N different members flag the same domain — without building a full intake form yet. Member-or-above can flag (broader than admin/owner since the action is informational).

  2. Deep link from /member-profile to /team — one-liner under "Brand identity" pointing at the team page where the registry hierarchy is now shown. Closes the "I came to member-profile to check if my company is classified right" bounce-off case.

Test plan

  • npm run typecheck clean
  • New integration test covers POST happy path + invalid-kind 400 + invalid-domain 400
  • 13/13 pass on the toggles suite

🤖 Generated with Claude Code

Two polish items deferred from the PR #3450 review (product expert
recommendations):

- Click-tracking endpoint POST /api/organizations/:orgId/brand-classification-report
  records {orgId, userId, kind, subject_domain} to registry_audit_log on
  every Report-wrong click. The mailto: opens regardless via the existing
  anchor href; sendBeacon doesn't block navigation. Gives us a triage
  queue + lets us detect when N different members all flag the same
  domain. Member-or-above can flag (broader than admin/owner since the
  action is informational and the corrective action is admin-side).
- One-line deep link from /member-profile's Brand identity section to
  the /team page where the registry hierarchy is now shown. Closes the
  "I came to member-profile to check if my company is classified right"
  bounce-off case.

1 new integration test (happy path + invalid-kind 400 + invalid-domain
400). 13/13 pass on the toggles suite.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@bokelley bokelley merged commit 9a664b6 into main Apr 28, 2026
13 checks passed
@bokelley bokelley deleted the bokelley/team-page-polish-and-deep-link branch April 28, 2026 18:59
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