Skip to content

Conversation

The-inside-man
Copy link
Contributor

No description provided.

Copy link
Contributor

@jaeopt jaeopt left a comment

Choose a reason for hiding this comment

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

master has optConfigV2 fix that is not released yet. We need release this fix from v3.9.0

@The-inside-man
Copy link
Contributor Author

master has optConfigV2 fix that is not released yet. We need release this fix from v3.9.0

I was told to do this PR, then create a new release branch from v3.9.0 and cherrypick this commit and the fix commit once ready. FSC has a few failures so need to make some changes first.

@coveralls
Copy link

coveralls commented Jul 14, 2021

Coverage Status

Coverage remained the same at 95.646% when pulling b08c3cb on jbrown/changelog-update into ab89cf4 on master.

@The-inside-man
Copy link
Contributor Author

The-inside-man commented Jul 15, 2021

Once merged in I will create a new branch off release V3.9.0 commit and cherry pick this and the other fix commit into the new release branch which will be what we release for the Python SDK.

@The-inside-man The-inside-man requested a review from jaeopt July 15, 2021 12:23
@@ -45,12 +45,12 @@ def create_impression_event(
if not activated_experiment and rule_type is not enums.DecisionSources.ROLLOUT:
return None

variation, experiment_key = None, None
variation, experiment_id = None, None
Copy link
Contributor

Choose a reason for hiding this comment

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

update license header

@The-inside-man The-inside-man deleted the jbrown/changelog-update branch July 15, 2021 16:53
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.

4 participants