Skip to content

fix: feature enablement in admin api#853

Open
ferhatelmas wants to merge 1 commit intomasterfrom
admin-features
Open

fix: feature enablement in admin api#853
ferhatelmas wants to merge 1 commit intomasterfrom
admin-features

Conversation

@ferhatelmas
Copy link
Member

What kind of change does this PR introduce?

Bug fix

What is the current behavior?

Feature check is done wrong so condition executes when it shouldn't.

What is the new behavior?

Condition runs correctly.

Additional context

Part of improvement in static analysis.

Signed-off-by: ferhat elmas <elmas.ferhat@gmail.com>
@coveralls
Copy link

Pull Request Test Coverage Report for Build 21818853226

Details

  • 3 of 3 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.006%) to 75.903%

Totals Coverage Status
Change from base Build 21777199007: 0.006%
Covered Lines: 26154
Relevant Lines: 34168

💛 - Coveralls

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