fix(dashboard): support custom-domain self-host onboarding - #806
izadoesdev wants to merge 6 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
The latest updates on your projects. Learn more about Unkey Deploy
|
a5acf33 to
94e13d9
Compare
Self-hosted production dashboards blocked requests to custom API origins, could not share sessions across explicitly configured subdomains, and generated tracking snippets that sent events to hosted ingestion. This change uses the configured origins and ingestion URL—including the copied AI installation prompt—honors
BETTER_AUTH_COOKIE_DOMAIN, and skips automatic verification email when verification is disabled.Rebased onto current
staging(c88bc6d1f), preserving the new hosted Dub attribution while disabling it in self-host mode. Signup keeps a sign-in link available.SELFHOST=trueomits the dashboard's first-party tracker and advertising/attribution scripts; Turbo includes that setting in its build environment and cache key. Hosted authentication, cookie security, verification, and telemetry defaults remain unchanged. Self-host signup also skips internal Dub and Slack notifications, and the build clears copied OpenAI pixel IDs so the direct registration handler cannot initialize tracking. The AI installation prompt reuses the existing React, Vue, and HTML snippet generators, removing duplicated options and cloud-only instructions.Validation
bun run lintandbun run check-typespass; the pre-push test hook completed all 27 tasks successfully.c88bc6d1f: all eight slices pass lint, all 33 typecheck tasks, all 27 root test tasks, and a production dashboard build. The built config setsNEXT_PUBLIC_SELFHOST=trueand clears a copied advertising key; the key is absent from browser JavaScript./selfhost-smoke, with no email, Autumn, AI, or Kafka credentials.Scope and overlaps
Based directly on
staging. The self-host quick start in #804 depends on this fix alongside #803 (email senders) and #805 (optional ingestion integrations). The auth changes touch the configuration surface also used by #656 and #692; dependency upgrades and session storage are unchanged.#816 handles local feature access without Autumn, and #814 keeps omitted self-host URL defaults local. When integrating #816, retain both CI test steps and both Next env entries (
NEXT_PUBLIC_SELFHOSTandNEXT_PUBLIC_OPENAI_ADS_PIXEL_ID).