Skip to content

fix: connection display names and gym row UX - #105

Merged
rghamilton3 merged 4 commits into
mainfrom
fix/connection-display-names-and-gym-row-ux
Apr 12, 2026
Merged

rghamilton3 merged 4 commits into
mainfrom
fix/connection-display-names-and-gym-row-ux

Conversation

@rghamilton3

Copy link
Copy Markdown
Contributor

Summary

  • Connections screen: Peer display names now resolve via useUserProfile instead of showing raw UUIDs. Falls back to 'Connected Athlete' when the peer's profile is private or has no display name set (respects display_visible RLS policy).
  • Gym management row: Gym name/address area is now a navigable link to the gym detail page. The Leave button is hidden for gym owners (who cannot leave their own gym). Test coverage updated to match.

Test plan

  • Connections screen shows display names for visible peers and 'Connected Athlete' for private profiles
  • Gym row navigates to gym detail page on tap
  • Leave button absent for gym owner, present for non-owner members
  • Existing gym-management-section tests pass

…MN syntax

- Rename 013_workout_note_tags.sql to 014_workout_note_tags.sql to fix
  duplicate version number conflict with 013_drop_display_visible.sql
- Remove IF EXISTS from ALTER TABLE DROP COLUMN in migration 013; SQLite
  does not support that syntax variant, causing a startup panic
Resolves connection list showing raw user UUIDs by resolving each peer
profile via the existing useUserProfile hook (gated by display_visible RLS).
Falls back to 'Connected Athlete' when the peer profile is private or has
no displayName set.
Wraps the gym name/address area in a Link to the gym detail page and
suppresses the Leave button for gym owners (who cannot leave their own gym).
Updates test coverage for the conditional leave button behavior.
@vercel

vercel Bot commented Apr 12, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ardent-forge Ready Ready Preview, Comment Apr 12, 2026 9:47pm

@rghamilton3
rghamilton3 merged commit 2661f55 into main Apr 12, 2026
4 checks passed

This branch was successfully deployed

1 active deployment
Preview — 54546bbe Deployed Apr 12, 2026 by vercel[bot]
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