Skip to content
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

prepare 5.2.0 release #107

Merged
merged 29 commits into from
Aug 30, 2018
Merged

prepare 5.2.0 release #107

merged 29 commits into from
Aug 30, 2018

Conversation

eli-darkly
Copy link
Contributor

[5.2.0] - 2018-08-29

Added:

  • The new LDClient method variation_detail allows you to evaluate a feature flag (using the same parameters as you would for variation) and receive more information about how the value was calculated. This information is returned in an EvaluationDetail object, which contains both the result value and a "reason" object which will tell you, for instance, if the user was individually targeted for the flag or was matched by one of the flag's rules, or if the flag returned the default value due to an error.

Fixed:

  • Evaluating a prerequisite feature flag did not produce an analytics event if the prerequisite flag was off.

ashanbrown and others added 29 commits July 23, 2018 17:09
log exceptions at error level and stacktraces at debug level
add new version of all_flags that captures more metadata
add ability to filter for only client-side flags
implement evaluation with explanations
fix event generation for a prerequisite that is off
Copy link
Contributor

@zsiddique zsiddique left a comment

Choose a reason for hiding this comment

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

Looks good but it seems FOSA is catching an LGPL license.

@eli-darkly
Copy link
Contributor Author

The FOSSA warning started prior to the last release, and isn't related to any code change as far as I can tell, so their own process must have changed. It's a development-only dependency, so not a priority at the moment - it would be good to make it go away but I'm not sure how much we can really do about it.

@eli-darkly eli-darkly merged commit 4adda57 into master Aug 30, 2018
@eli-darkly eli-darkly deleted the 5.2.0 branch August 30, 2018 18:28
eli-darkly added a commit that referenced this pull request Mar 21, 2019
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