Skip to content

isfeatureenabled should return boolean value. #39

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

Merged

Conversation

msohailhussain
Copy link
Contributor

isFeatureEnabled must return only boolean value. NULL is replaced with false.
@mikeng13

@optibot
Copy link
Contributor

optibot commented Nov 21, 2017

Can one of the admins verify this patch?

Copy link
Contributor

@mikeproeng37 mikeproeng37 left a comment

Choose a reason for hiding this comment

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

lgtm

@kellyroach-optimizely
Copy link
Contributor

157/157 unit tests PASSED and the code changes look good.

@kellyroach-optimizely kellyroach-optimizely merged commit 97ac579 into optimizely:master Nov 27, 2017
kellyroach-optimizely added a commit that referenced this pull request Dec 12, 2017
kellyroach-optimizely added a commit that referenced this pull request Dec 20, 2017
* *.nuspec more like it was in 1.2.0 .  Some improvements survive.

* pack.ps1 only packages Optimizely's *.dll's

* Revert "isfeatureenabled should return boolean value. (#39)"

This reverts commit 97ac579.

* 'Feature Flags & Rollout' not included in 1.3.0

* 'Feature Flags & Rollout' DecisionService.cs code not included in 1.3.0

* notification center property in optimizely class changed into public access modifier. (#40)

* GetVariation should return variation object. (#41)

* Revert "GetVariation should return variation object. (#41)"

This reverts commit 3740441.

* Update CHANGELOG.md

* Update CHANGELOG.md '1.3.0' release date
@mfahadahmed mfahadahmed deleted the is_feature_enabled_fix branch March 7, 2018 07:26
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.

5 participants