-
Notifications
You must be signed in to change notification settings - Fork 101
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
chore(config): move more remote config to Statsig #6474
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6474 +/- ##
==========================================
+ Coverage 89.02% 89.04% +0.02%
==========================================
Files 729 729
Lines 31871 31858 -13
Branches 5819 6138 +319
==========================================
- Hits 28373 28369 -4
+ Misses 3450 3441 -9
Partials 48 48
... and 3 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
Description
This PR moves the following remote config variables to statsig dynamic config:
maxNumRecentDapps
is no longer used and just removed hereTest plan
n/a
Related issues
Backwards compatibility
Y
Network scalability
If a new NetworkId and/or Network are added in the future, the changes in this PR will: