Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(statsig-adapter): Release Statsig Adapter #43

Open
wants to merge 24 commits into
base: main
Choose a base branch
from
Open

Conversation

AAorris
Copy link
Contributor

@AAorris AAorris commented Jan 20, 2025

Overview

Statsig has client | server | console api keys. We can shore up the naming with statsig, [type], and api key. Use this convention across the adapter.

Move Edge Config related dependencies to peer dependencies and dynamically import them when used, so that the flags SDK can be used without including Vercel if it's not needed.

Constrain the package.json requests to the major version, and let the app pin versions if desired

@AAorris AAorris requested review from dferber90 and AndyBitz January 20, 2025 21:14
@AAorris AAorris self-assigned this Jan 20, 2025
Copy link

vercel bot commented Jan 20, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
flags-sdk-snippets ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 19, 2025 7:34pm
summer-sale ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 19, 2025 7:34pm

Add experimentation primitives, disable exposure logging by default, identify StatsigUser instead of `{ statsigUser: StatsigUser }`
@AAorris AAorris changed the title (statsig-adapter): Move Edge Config to peer deps, more consistent api key naming (statsig-adapter): Release Statsig Adapter Feb 18, 2025
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.

2 participants