[FSSDK-10553] add IsEveryoneElseVariation marker in decide API

optimizely / agent

Agent service for Optimizely Feature Experimentation and Optimizely Full Stack (legacy)

Apache License 2.0

31 stars 26 forks source link

[FSSDK-10553] add IsEveryoneElseVariation marker in decide API #422

Closed pulak-opti closed 1 month ago

pulak-opti commented 1 month ago

Summary

added IsEveryoneElseVariation marker in decide API

Ticket

FSSDK-10553

pulak-opti commented 1 month ago

Hey @mikechu-optimizely Thanks for your thoughts. I have tried to implement this in the SDK first. You might have noticed the linked closed PR in SDK. But then I have changed my mind for the followings:

User has requested the feature for the Agent
Implementing this in SDK will force us to add this in other primary language SDKs to keep things in sync.
Users can implement the same thing on their server side in the similar way as we have done it in Agent if they need it.

jaeopt commented 1 month ago

@mikechu-optimizely @pulak-opti I also see some value for adding this to all SDKs, hiding our naming rule for everyone-else. The current implementation in agent looks reasonable for now and we can review it again for adding into the SDK level.