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

Use UNSET_VALUE as default for relation of severity values in powerfilter #3052

Merged
merged 3 commits into from
Jul 2, 2021

Conversation

swaterkamp
Copy link
Member

What:
Use UNSET_VALUE as default for relation of severity values in powerfilter

Why:
The Select showed "is equal to" as default value, but when updating the filter, this setting was not applied. The displayed item is set to "--" (UNSET_VALUE) to prevent confusion.

How:
Manual testing - explicitly whether/which default value is applied when not changing values in the filterdialog. Also adjusted automated tests.

Checklist:

  • Tests
  • CHANGELOG Entry
  • Labels for ports to other branches

@swaterkamp swaterkamp added port-to-stable Use mergifiy to port PR to stable port-to-21.10 labels Jul 2, 2021
@swaterkamp swaterkamp requested a review from sarahd93 July 2, 2021 14:52
@swaterkamp swaterkamp self-assigned this Jul 2, 2021
@swaterkamp swaterkamp requested a review from a team as a code owner July 2, 2021 14:52
@codecov
Copy link

codecov bot commented Jul 2, 2021

Codecov Report

Merging #3052 (8c47c49) into gsa-20.08 (f48e62c) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           gsa-20.08    #3052   +/-   ##
==========================================
  Coverage      53.74%   53.74%           
==========================================
  Files           1073     1073           
  Lines          25912    25912           
  Branches        7375     7375           
==========================================
  Hits           13926    13926           
  Misses         10882    10882           
  Partials        1104     1104           
Impacted Files Coverage Δ
...src/web/components/powerfilter/relationselector.js 100.00% <100.00%> (ø)
.../web/components/powerfilter/severityvaluesgroup.js 91.66% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f48e62c...8c47c49. Read the comment docs.

@swaterkamp swaterkamp enabled auto-merge July 2, 2021 15:04
@swaterkamp swaterkamp merged commit e3fe1e2 into gsa-20.08 Jul 2, 2021
@swaterkamp swaterkamp deleted the swaterkamp/severityRelation branch July 2, 2021 17:43
y0urself added a commit that referenced this pull request Jul 3, 2021
Use UNSET_VALUE as default for relation of severity values in powerfilter (backport #3052)
y0urself added a commit that referenced this pull request Jul 3, 2021
Use UNSET_VALUE as default for relation of severity values in powerfilter (backport #3052)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
port-to-stable Use mergifiy to port PR to stable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants