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

FFM-8670: Feature flag rule not being evaluated for a given attribute #27

Merged
merged 1 commit into from
Jul 19, 2023

Conversation

ribeirophillipe
Copy link
Contributor

@ribeirophillipe ribeirophillipe commented Jul 19, 2023

Fix evaluation issue where a boolean is returned instead of a value. In the issue FFM-8670, the customer reported that flag was still returning the same true value, even though the flag was set and that boolean is compared against the list of IN clause values from the flag’s config, which always fails.

@ribeirophillipe ribeirophillipe self-assigned this Jul 19, 2023
@ribeirophillipe ribeirophillipe force-pushed the FFM-8670 branch 7 times, most recently from 0f0b044 to e5aa9ba Compare July 19, 2023 03:53
* Fix evaluation issue where a boolean is returned instead of a value
* Include test case
Copy link
Contributor

@andybharness andybharness 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 - might want to update the PR/commit message with more detail just so anyone browsing the history can get an idea of what was fixed.

@ribeirophillipe ribeirophillipe merged commit c108acf into main Jul 19, 2023
@ribeirophillipe ribeirophillipe deleted the FFM-8670 branch July 19, 2023 14:20
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.

2 participants