Skip to content

feat(expert-consultation): add banner and modal with HubSpot form integration#1769

Open
samuv wants to merge 3 commits intonewsletter-modalfrom
talk-to-expert
Open

feat(expert-consultation): add banner and modal with HubSpot form integration#1769
samuv wants to merge 3 commits intonewsletter-modalfrom
talk-to-expert

Conversation

@samuv
Copy link
Collaborator

@samuv samuv commented Mar 19, 2026

This is a stacked PR that need to be merged after this

Adds an expert consultation banner shown on group routes when a group has more than 3 servers. Clicking the CTA opens a modal with a HubSpot form. Submitting suppresses permanently, dismissing suppresses for 30 days. Does not overlap with the newsletter modal.

  • Add expert-consultation electron-store and SQLite dual-write for expertConsultationSubmitted and expertConsultationDismissedAt, reconciled on startup
  • Expose getExpertConsultationState, setExpertConsultationSubmitted, setExpertConsultationDismissedAt via IPC and preload API
  • Add ExpertConsultationBanner component with banner visibility logic (route, server count, newsletter overlap, dismiss/submit state)
  • Add ExpertConsultationDialog with Zod validation, HubSpot v3 API submission (form 5f1a7a2c-5069-44b7-9444-d952c55ce89c), consent handling, success/error states, Sentry capture, and analytics
  • Mount banner in root layout, scoped to /group/$groupName routes
  • Add unit and component tests for state persistence, reconciliation, and banner/modal behavior
Kapture.2026-03-19.at.15.34.09.mp4

@samuv samuv self-assigned this Mar 19, 2026
@samuv samuv linked an issue Mar 19, 2026 that may be closed by this pull request
5 tasks
@samuv
Copy link
Collaborator Author

samuv commented Mar 19, 2026

/build-test

@github-actions
Copy link
Contributor

Build Artifacts for PR #1769

Platform Architecture Status
macOS arm64 ✅ Ready
macOS x64 ✅ Ready
Windows arm64 ✅ Ready
Windows x64 ✅ Ready
Linux arm64 ✅ Ready
Linux x64 ✅ Ready

Download artifacts from workflow run

Version: pr-1769 | Artifacts expire in 7 days | Triggered by @samuv

Note: macOS builds are signed (not notarized). Windows and Linux builds are unsigned.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Task] Onramp: Enterprise demo request popup

2 participants