Skip to content

feat: hide and show pylon on user login events#7383

Merged
Zaimwa9 merged 2 commits intomainfrom
fix/reactive-pylon-show-hide
Apr 30, 2026
Merged

feat: hide and show pylon on user login events#7383
Zaimwa9 merged 2 commits intomainfrom
fix/reactive-pylon-show-hide

Conversation

@Zaimwa9
Copy link
Copy Markdown
Contributor

@Zaimwa9 Zaimwa9 commented Apr 30, 2026 •
edited
Loading

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

Logging in with a paid account then logging out and in with a free user would keep the bubble chat visible. As well as the opposite way.

  • Make the visibility reactive by calling identifyChatUser when switching org and logging out

How did you test this code?

Steps to reproduce:

  • Login with a paid account
  • Logout (pylon was still visible)

@Zaimwa9 Zaimwa9 requested a review from a team as a code owner April 30, 2026 14:10
@Zaimwa9 Zaimwa9 requested review from kyle-ssg and removed request for a team April 30, 2026 14:10
claude[bot]
claude Bot reviewed Apr 30, 2026
View reviewed changes
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

⚠️ Code review skipped — your organization's overage spend limit has been reached.

Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.

Once credits are available, reopen this pull request to trigger a review.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 30, 2026 •
edited
Loading

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

Project Deployment Actions Updated (UTC)
flagsmith-frontend-preview Ready Ready Preview, Comment Apr 30, 2026 2:16pm
flagsmith-frontend-staging Ready Ready Preview, Comment Apr 30, 2026 2:16pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Ignored Ignored Preview Apr 30, 2026 2:16pm

Request Review

@Zaimwa9 Zaimwa9 requested review from talissoncosta and removed request for kyle-ssg April 30, 2026 14:10
@github-actions github-actions Bot added front-end Issue related to the React Front End Dashboard feature New feature or request labels Apr 30, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 30, 2026 •
edited
Loading

Docker builds report

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

talissoncosta
talissoncosta previously approved these changes Apr 30, 2026
View reviewed changes
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 30, 2026 •
edited
Loading

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

passed  11 passed

Details

stats  11 tests across 8 suites
duration  51.8 seconds
commit  54d02a3
info  🔄 Run: #16473 (attempt 1)

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

passed  11 passed

Details

stats  11 tests across 8 suites
duration  44 seconds
commit  54d02a3
info  🔄 Run: #16473 (attempt 1)

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

passed  2 passed

Details

stats  2 tests across 2 suites
duration  47.2 seconds
commit  54d02a3
info  🔄 Run: #16473 (attempt 1)

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

passed  2 passed

Details

stats  2 tests across 2 suites
duration  1 minute, 3 seconds
commit  54d02a3
info  🔄 Run: #16473 (attempt 1)

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

passed  11 passed

Details

stats  11 tests across 8 suites
duration  43.9 seconds
commit  8b7da20
info  🔄 Run: #16474 (attempt 1)

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

passed  11 passed

Details

stats  11 tests across 8 suites
duration  49.4 seconds
commit  8b7da20
info  🔄 Run: #16474 (attempt 1)

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

passed  2 passed

Details

stats  2 tests across 2 suites
duration  53.8 seconds
commit  8b7da20
info  🔄 Run: #16474 (attempt 1)

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

passed  2 passed

Details

stats  2 tests across 2 suites
duration  1 minute, 32 seconds
commit  8b7da20
info  🔄 Run: #16474 (attempt 1)

@github-actions github-actions Bot added feature New feature or request and removed feature New feature or request labels Apr 30, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 30, 2026 •
edited
Loading

Visual Regression

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

@Zaimwa9 Zaimwa9 requested a review from talissoncosta April 30, 2026 14:20
@Zaimwa9 Zaimwa9 merged commit 85e994d into main Apr 30, 2026
30 checks passed
@Zaimwa9 Zaimwa9 deleted the fix/reactive-pylon-show-hide branch April 30, 2026 14:31
@flagsmithdev flagsmithdev mentioned this pull request Apr 30, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@claude claude[bot] claude[bot] left review comments

@talissoncosta talissoncosta talissoncosta approved these changes

Assignees

No one assigned

Labels

feature New feature or request front-end Issue related to the React Front End Dashboard

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Zaimwa9 @talissoncosta