Remove ConfigCat and Amplitude integrations - #195
Merged
Conversation
- Remove configcat-js-ssr and @amplitude/analytics-node dependencies - Delete A/B testing infrastructure (variants, content, configcat client) - Hardcode variant A content directly in Hero and Features components - Remove middleware (was only for user tracking) - Remove amplitude events API route - Simplify EventLink to use only Vercel Analytics Amp-Thread-ID: https://ampcode.com/threads/T-019c00e7-1e04-742b-aded-3416acb020cd Co-authored-by: Amp <amp@ampcode.com>
carsonfarmer
requested review from
dtbuchholz,
joewagner and
sanderpick
as code owners
January 27, 2026 19:31
- actions/checkout v2 -> v4 - actions/setup-node v1 -> v4 with built-in caching - actions/cache v1 removed (now handled by setup-node) - Node 16 -> 20 Amp-Thread-ID: https://ampcode.com/threads/T-019c00e7-1e04-742b-aded-3416acb020cd Co-authored-by: Amp <amp@ampcode.com>
dtbuchholz
previously approved these changes
Jan 27, 2026
dtbuchholz
left a comment
dtbuchholz
left a comment
Contributor
There was a problem hiding this comment.
not sure why the vercel deployment is failing...but i can't log in to verify. maybe it's fine?
i ran the site locally, and it worked fine.
Contributor
|
Vercel build fails because our version of node is too old. |
Amp-Thread-ID: https://ampcode.com/threads/T-019c00e7-1e04-742b-aded-3416acb020cd Co-authored-by: Amp <amp@ampcode.com>
Member
Author
|
Ok, I updated the "suggested" node version in package.json, and then actually updated in our vercel deployment settings. Build looks good, you can checkout the preview deployment now. Once this is merged, we can delete config-cat and amplitude accounts. |
dtbuchholz
approved these changes
Jan 28, 2026
joewagner
approved these changes
Jan 28, 2026
joewagner
left a comment
joewagner
left a comment
Contributor
There was a problem hiding this comment.
Looks good to me. Only nit would be to update to node 22 instead of 20, so we don't have to update again when 20 times out, but I'd say merge it.
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
3 participants
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.
Amp-Thread-ID: https://ampcode.com/threads/T-019c00e7-1e04-742b-aded-3416acb020cd