{{ message }}
Repository navigation
feat(theme): load bootstrap.js on docs.apify.com for signup attribution - #3099
Open
webrdaniel wants to merge 4 commits into
Open
webrdaniel wants to merge 4 commits into
webrdaniel wants to merge 4 commits into
Conversation
Sets the first-touch and FirstPromoter cookies on .apify.com that Console reads at signup. Only production builds load it, so previews and local builds write none of them. Closes #3085 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ap.js Preview hosts proxy the SDK, client and CLI paths to their production builds, so the build-time check still loaded the script there and wrote cookies to .apify.com. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
apify-service-account
added
the
t-web
Contributor
webrdaniel
requested review from
birosrichard,
katzino and
mr-rajce
and
a balanced review from Copilot
October 21, 2026 11:18
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The narrowly scoped change has no unresolved findings, and focused reviewer checks confirmed the hostname guard excludes preview and local hosts.
No open issues 🎉
What changed in this PR
Adds signup attribution to the shared docs theme so visits starting on docs can be recorded by Console.
Changes:
- Injects
https://apify.com/ext/bootstrap.jsinto the page head. - Loads it only on
docs.apify.com, excluding preview and local hosts.
Review effort: Balanced
Previews get a handful of real visitors who later sign up, so the hostname check wasn't worth the extra code. Local builds still can't write the cookies, because browsers reject apify.com cookies from other hosts. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
katzino
approved these changes
Oct 21, 2026
birosrichard
approved these changes
Oct 21, 2026
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Loads
apify.com/ext/bootstrap.json all six docs sites, so visitors who land on docs first get the first-touch and FirstPromoter cookies that Console reads at signup.Closes #3085. Came up in this #acquisition-team-collaboration thread.
Worth knowing before merge
docs.apify.com, and a docs page as their first URL. The data team needs the date.fpr, for example on old/platform/...links. That's a separate PR.apify.comcookies from other hosts.ext/bootstrap.js.Tested live on docs.apify.com: all four cookies set on
.apify.com, and apify.com reads them.🤖 Generated with Claude Code