Skip to content

Comments

chore: e2e Android sample - move keys and url into config#382

Merged
abelonogov-ld merged 3 commits intomainfrom
andrey/friendly-config
Feb 19, 2026
Merged

chore: e2e Android sample - move keys and url into config#382
abelonogov-ld merged 3 commits intomainfrom
andrey/friendly-config

Conversation

@abelonogov-ld
Copy link
Contributor

@abelonogov-ld abelonogov-ld commented Feb 19, 2026

Summary

  • Move keys into config to reduce change of accidental key merge
  • add flagEvaluation method on the app launch for further investigations

Note

Low Risk
Build-time config wiring plus additional debug logging/listeners in an e2e sample; main risk is misconfiguration (empty local.properties values) causing the app to run without a valid SDK key or expected endpoints.

Overview
The Android e2e sample now reads LaunchDarkly configuration from local.properties and exposes it via BuildConfig (LAUNCHDARKLY_MOBILE_KEY, OTLP_ENDPOINT, BACKEND_URL), replacing the previously hardcoded mobile key and wiring the endpoints into ObservabilityOptions.

On app init, BaseApplication now performs a boolVariation for feature1 and registers a FeatureFlagChangeListener to log subsequent changes, aiding investigations and validating flag updates.

Written by Cursor Bugbot for commit fa2b882. This will update automatically on new commits. Configure here.

@abelonogov-ld abelonogov-ld requested a review from a team as a code owner February 19, 2026 21:21
@abelonogov-ld abelonogov-ld enabled auto-merge (squash) February 19, 2026 22:09
@abelonogov-ld abelonogov-ld merged commit a996b19 into main Feb 19, 2026
22 of 23 checks passed
@abelonogov-ld abelonogov-ld deleted the andrey/friendly-config branch February 19, 2026 22:40
abelonogov-ld added a commit that referenced this pull request Feb 20, 2026
* main:
  chore: e2e Android sample - move keys and url into config (#382)
  chore: Update root documentation to represent all packages (#377)
  chore: release main (#378)
  fix: Show correct import in documentation (#376)
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