FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Detect changes in AB Testing (ABT) experiment metadata during config fetch by athiramanu · Pull Request #8002 · firebase/firebase-android-sdk · GitHub

Detect changes in AB Testing (ABT) experiment metadata during config fetch - #8002

Merged
athiramanu merged 6 commits into
mainfrom
realtime-abt
Apr 10, 2026
Merged

Detect changes in AB Testing (ABT) experiment metadata during config fetch#8002
athiramanu merged 6 commits into
mainfrom
realtime-abt

Conversation

Copy link
Copy Markdown
Contributor

Updated to detect changes in AB Testing (ABT) experiment metadata. This ensures that if an experiment's variant or affected parameter keys change, the associated config keys are correctly identified as changed, even if the parameter values remain the same.

Key changes:

  • Added constant to .
  • Implemented in to map affected config keys to their respective experiment descriptions.
  • Modified to compare experiment metadata for each config key.
  • Updated with cases for changed, deleted, and key-shifted experiments.

Validation:

  • Adding an experiment with 100% exposure assigns user to a variant
  • Updating exposure to 0% returns default value to user
  • Updating exposure back to 100% returns the previously assigned variant
  • Updating the variant value returns the new value to user

Testing on sample Android app: Link

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

github-actions Bot commented Mar 31, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

📝 PRs merging into main branch

Our main branch should always be in a releasable state. If you are working on a larger change, or if you don't want this change to see the light of the day just yet, consider using a feature branch first, and only merge into the main branch when the code complete and ready to be released.

tusharkhandelwal8 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Can we fix CI failures?

Updated  to detect changes in AB Testing (ABT) experiment metadata. This ensures that if an experiment's variant or affected parameter keys change, the associated config keys are correctly identified as changed, even if the parameter values remain the same.

Key changes:
- Added  constant to .
- Implemented  in  to map affected config keys to their respective experiment descriptions.
- Modified  to compare experiment metadata for each config key.
- Updated  with cases for changed, deleted, and key-shifted experiments.
athiramanu removed the request for review from ashish-kothari April 6, 2026 14:44
Comment thread firebase-config/CHANGELOG.md Outdated
athiramanu requested a review from rlazo April 9, 2026 00:21
athiramanu merged commit d40d90b into main Apr 10, 2026
33 checks passed
athiramanu deleted the realtime-abt branch April 10, 2026 04:52
github-actions Bot mentioned this pull request Apr 10, 2026
emilypgoogle added a commit that referenced this pull request Apr 22, 2026
Adds daily test status to the Test Reporting bug. Unlike unit and
instrumentation tests, all runs are always shown, since we currently
only have 1 type of daily test.

Example output below.

----- 

### Unit Tests

| | [#8009](#8009)
| [#7983](#7983) |
[#8002](#8002) |
[#8027](#8027) |
[#8028](#8028) |
[#8025](#8025) |
Success Rate |
| :--- | :---: | :---: | :---: | :---: | :---: | :---: | :--- |
| firebase-dataconnect | | | |
[✅](https://github.com/firebase/firebase-android-sdk/actions/runs/24202345424/job/70649528040)
| |
[⛔](https://github.com/firebase/firebase-android-sdk/actions/runs/24162429246/job/70516856982)
| ⛔ 50% |

*+8 passing SDKs
### Instrumentation Tests

*All tests passing*

### Daily Tests

| | Apr 15 | Apr 16 | Apr 17 | Apr 18 | Apr 19 | Apr 20 | Apr 21 |
Success Rate |
| :--- | :---: | :---: | :---: | :---: | :---: | :---: | :---: | :--- |
| AI Daily Tests |
[✅](https://github.com/firebase/firebase-android-sdk/actions/runs/24442505713)
|
[✅](https://github.com/firebase/firebase-android-sdk/actions/runs/24498252859)
|
[✅](https://github.com/firebase/firebase-android-sdk/actions/runs/24553833385)
|
[✅](https://github.com/firebase/firebase-android-sdk/actions/runs/24599892175)
|
[✅](https://github.com/firebase/firebase-android-sdk/actions/runs/24623963356)
|
[✅](https://github.com/firebase/firebase-android-sdk/actions/runs/24654541730)
|
[✅](https://github.com/firebase/firebase-android-sdk/actions/runs/24710324995)
| ✅ 100% |
firebase locked and limited conversation to collaborators May 10, 2026
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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL