Skip to content

Pre-fill visitor data in Lobbyside widget#3850

Merged
sarupbanskota merged 1 commit into
mainfrom
lobbyside-set-visitor-api
Apr 16, 2026
Merged

Pre-fill visitor data in Lobbyside widget#3850
sarupbanskota merged 1 commit into
mainfrom
lobbyside-set-visitor-api

Conversation

@sarupbanskota

Copy link
Copy Markdown
Member

Summary

  • Calls Lobbyside.setVisitor() with the current user's email, name, and GitHub username after the widget script loads
  • When a user clicks "Join" on the Lobbyside widget, their info is pre-filled in the lobby form so they don't have to type it again
  • Also syncs data if the script was already loaded (e.g., navigating back to a page with the widget)

Test plan

  • Log in as a staff user, visit a page with the Lobbyside widget
  • Click "Join" on the widget card
  • Verify the lobby form has email, name, and GitHub handle pre-filled
  • Verify fields are still editable

🤖 Generated with Claude Code

Call Lobbyside.setVisitor() with the current user's email, name,
and GitHub username so the lobby form is pre-filled when they join.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@sarupbanskota sarupbanskota merged commit 7b40c31 into main Apr 16, 2026
1 of 4 checks passed
@sarupbanskota sarupbanskota deleted the lobbyside-set-visitor-api branch April 16, 2026 04:39
@codecov

codecov Bot commented Apr 16, 2026

Copy link
Copy Markdown

Bundle Report

Changes will increase total bundle size by 249 bytes (0.0%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
client-array-push 39.47MB 249 bytes (0.0%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: client-array-push

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/chunk.*.js 249 bytes 3.44MB 0.01%

@codecov

codecov Bot commented Apr 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 45.45455% with 6 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
app/components/lobbyside-widget/index.ts 45.45% 2 Missing and 4 partials ⚠️

❌ Your patch status has failed because the patch coverage (45.45%) is below the target coverage (100.00%). You can increase the patch coverage or adjust the target coverage.

📢 Thoughts on this report? Let us know!

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