Skip to content

fix: add navigation to identity page from segment members list#7927

Merged
kyle-ssg merged 1 commit into
mainfrom
fix/segment-members-navigation
Jul 8, 2026
Merged

fix: add navigation to identity page from segment members list#7927
kyle-ssg merged 1 commit into
mainfrom
fix/segment-members-navigation

Conversation

@kyle-ssg

@kyle-ssg kyle-ssg commented Jul 1, 2026

Copy link
Copy Markdown
Member

Thanks for submitting a PR! Please check the boxes below:

  • I have read the Contributing Guide.
  • I have added information to docs/ if required so people know about the feature.
  • I have filled in the "Changes" section below.
  • I have filled in the "How did you test this code" section below.

Changes

Fixes a url link in the segment identities page.

How did you test this code?

Visit a segment and click on the identities tab, click one of the rows

- Add Link to each identity row in SegmentMembersList
- Pass environmentApiKey from parent component for URL building
- Use identity-id route format which only requires identifier

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@kyle-ssg kyle-ssg requested a review from a team as a code owner July 1, 2026 12:16
@kyle-ssg kyle-ssg requested review from talissoncosta and removed request for a team July 1, 2026 12:16
@vercel

vercel Bot commented Jul 1, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
flagsmith-frontend-preview Ready Ready Preview, Comment Jul 1, 2026 12:17pm
flagsmith-frontend-staging Ready Ready Preview, Comment Jul 1, 2026 12:17pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Ignored Ignored Jul 1, 2026 12:17pm

Request Review

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Docker builds report

Image Build Status Security report
ghcr.io/flagsmith/flagsmith-api-test:pr-7927 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-e2e:pr-7927 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-private-cloud:pr-7927 Finished ✅ Results
ghcr.io/flagsmith/flagsmith-api:pr-7927 Finished ✅ Results
ghcr.io/flagsmith/flagsmith:pr-7927 Finished ✅ Results
ghcr.io/flagsmith/flagsmith-frontend:pr-7927 Finished ✅ Results

@github-actions github-actions Bot added the fix label Jul 1, 2026
@kyle-ssg kyle-ssg requested a review from Zaimwa9 July 1, 2026 12:18
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Playwright Test Results (oss - depot-ubuntu-latest-16)

passed  5 passed

Details

stats  5 tests across 3 suites
duration  40.6 seconds
commit  46bb280
info  🔄 Run: #17994 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

passed  4 passed

Details

stats  4 tests across 3 suites
duration  45.7 seconds
commit  46bb280
info  🔄 Run: #17994 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

passed  3 passed

Details

stats  3 tests across 3 suites
duration  58.5 seconds
commit  46bb280
info  🔄 Run: #17994 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

passed  2 passed

Details

stats  2 tests across 2 suites
duration  47 seconds
commit  46bb280
info  🔄 Run: #17994 (attempt 1)

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Visual Regression

17 screenshots compared. See report for details.
View full report

@kyle-ssg kyle-ssg merged commit f005081 into main Jul 8, 2026
30 of 31 checks passed
@kyle-ssg kyle-ssg deleted the fix/segment-members-navigation branch July 8, 2026 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix front-end Issue related to the React Front End Dashboard

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants