Skip to content

[FSSDK-11446] update: experiment_id and variation_id added to notification listener payloads #1030

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

FarhanAnjum-opti
Copy link
Contributor

Summary

  • Added experiment_id and variation_id to the decision notification listener payload in the Python SDK.
  • Ensured that these IDs are included in all decision notifications sent via the notification center.
  • Updated unit tests to validate the presence of experiment_id and variation_id in the payload.

Why

To support data warehouse-native customers who rely on SDK notification listeners to send experiment data to their data warehouse, we need to include experiment_id and variation_id in the payload. This ensures that the stats engine can correctly compute experiment results based on warehouse data.

Test Plan

  • Modify existing or add unit tests to check for the presence of experiment_id and variation_id in the notification payload.
  • Run all tests to confirm that the changes do not introduce regressions.

Issues

-fixed unit tests to expect experiment id and variation id in notification listener payload
@coveralls
Copy link

coveralls commented Apr 23, 2025

Coverage Status

coverage: 80.74% (+0.03%) from 80.714%
when pulling e2cfa9a on farhan-anjum/FSSDK-11446-add-exp/var-id-to-notif-listener
into 0237be2 on master.

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.

3 participants