Skip to content

[AI-FSSDK] [FSSDK-12369] Add local holdouts support to C# SDK#408

Merged
junaed-optimizely merged 4 commits into
masterfrom
ai/mat001/FSSDK-12369-local-holdouts
Jun 3, 2026
Merged

[AI-FSSDK] [FSSDK-12369] Add local holdouts support to C# SDK#408
junaed-optimizely merged 4 commits into
masterfrom
ai/mat001/FSSDK-12369-local-holdouts

Conversation

@Mat001
Copy link
Copy Markdown
Contributor

@Mat001 Mat001 commented May 14, 2026

Summary

Adds local holdouts support to the C# SDK, enabling targeting of specific experiment or delivery rules within a flag via a new includedRules field on the Holdout model. Global holdouts (where includedRules is null) continue to apply to all rules, preserving backward compatibility.

Changes

  • Added includedRules field to the Holdout model to support local holdout targeting
  • Updated holdout evaluation logic to apply holdouts only to matching rules when includedRules is specified
  • Extended datafile deserialization to handle the new optional field

Jira Ticket

FSSDK-12369

@Mat001 Mat001 requested a review from junaed-optimizely May 18, 2026 17:14
Copy link
Copy Markdown
Contributor

@junaed-optimizely junaed-optimizely left a comment

Choose a reason for hiding this comment

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

I have made some necessary changes. Looks good to me overall. @muzahidul-opti could you please give a final look ?

Copy link
Copy Markdown

@muzahidul-opti muzahidul-opti left a comment

Choose a reason for hiding this comment

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

Local HO changes looks good to me.

@junaed-optimizely junaed-optimizely merged commit 30ccea9 into master Jun 3, 2026
9 of 11 checks passed
@junaed-optimizely junaed-optimizely deleted the ai/mat001/FSSDK-12369-local-holdouts branch June 3, 2026 13:07
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