-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
docs(dart): Add feature flags documentation #13391
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
base: master
Are you sure you want to change the base?
Conversation
@denrase is attempting to deploy a commit to the Sentry Team on Vercel. A member of the Team first needs to authorize it. |
@buenaflor Think we are rdy for the vercel run? |
- Adding the `SentryFirebaseRemoteConfig` integration will automatically track feature flag evaluations for boolean Firebase Remote Config values. | ||
- Only boolean values are supported. |
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.
- Adding the `SentryFirebaseRemoteConfig` integration will automatically track feature flag evaluations for boolean Firebase Remote Config values. | |
- Only boolean values are supported. | |
- Adding the `SentryFirebaseRemoteConfig` integration will automatically track feature flag evaluations. | |
- Only boolean values are supported. |
```dart | ||
import 'package:sentry_flutter/sentry_flutter.dart'; | ||
import 'package:sentry_firebase_remote_config/sentry_firebase_remote_config.dart'; | ||
|
||
... // Initialize Firebase Remote Config | ||
|
||
await SentryFlutter.init( | ||
(options) { | ||
options.dsn = 'https://[email protected]/example'; | ||
options.addIntegration( | ||
SentryFirebaseRemoteConfigIntegration( | ||
firebaseRemoteConfig: firebaseRemoteConfig, | ||
// Don't call `await remoteConfig.activate();` when firebase config is updated. Per default this is true. | ||
activateOnConfigUpdated: false, | ||
), | ||
); | ||
}, | ||
); | ||
``` |
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.
imo we can remove the // Initialize Firebase Remote Config comment, the import and the options.dsn
description: With Feature Flags, Sentry tracks feature flag evaluations in your application, keeps an audit log feature flag changes, and reports any suspicious updates that may have caused an error. | ||
--- | ||
|
||
<PlatformContent includePath="feature-flags/prerelease-alert" /> |
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.
what does this do?
Generally looks good to me, since this is part of v9 I won't approve until we've GA'd |
DESCRIBE YOUR PR
Adds docs for Flutter Feature Flag implementation, which will be available in the upcoming v9 release.
IS YOUR CHANGE URGENT?
Help us prioritize incoming PRs by letting us know when the change needs to go live.
SLA
Thanks in advance for your help!
PRE-MERGE CHECKLIST
Make sure you've checked the following before merging your changes:
EXTRA RESOURCES