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

Fix changing Trend and Select for NVT-families and whole selection only #2905

Merged
merged 2 commits into from
May 12, 2021

Conversation

swaterkamp
Copy link
Member

@swaterkamp swaterkamp commented May 10, 2021

What:
Automatically send trend:0 (static) when a whole-selection only family is unselected.

Needs greenbone/gvmd#1517

Why:
gvmd needs an explicit trend:0 with an unselected whole-selection family

How:
Observe automatic radio button changes when selecting/unselecting a whole-selection family, verify that the old behavior still works for all other families (no automatic changes), and check parameters in network request.

Checklist:

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

@swaterkamp swaterkamp self-assigned this May 10, 2021
@codecov
Copy link

codecov bot commented May 10, 2021

Codecov Report

Merging #2905 (f8d79d3) into gsa-21.04 (74a1fd3) will increase coverage by 0.00%.
The diff coverage is 77.77%.

Impacted file tree graph

@@            Coverage Diff             @@
##           gsa-21.04    #2905   +/-   ##
==========================================
  Coverage      53.83%   53.84%           
==========================================
  Files           1075     1075           
  Lines          26226    26250   +24     
  Branches        7491     7506   +15     
==========================================
+ Hits           14118    14133   +15     
- Misses         10993    11000    +7     
- Partials        1115     1117    +2     
Impacted Files Coverage Δ
gsa/src/web/pages/notes/dialog.js 4.87% <ø> (ø)
gsa/src/web/pages/nvts/solution.js 22.22% <0.00%> (-11.12%) ⬇️
gsa/src/web/pages/radius/dialog.js 66.66% <ø> (ø)
gsa/src/web/pages/results/dashboard/index.js 75.00% <ø> (ø)
gsa/src/web/pages/results/details.js 6.74% <0.00%> (-0.16%) ⬇️
gsa/src/web/pages/targets/details.js 10.00% <ø> (ø)
gsa/src/web/pages/usersettings/dialog.js 29.41% <ø> (ø)
gsa/src/web/pages/scanners/dialog.js 70.31% <66.66%> (+0.47%) ⬆️
gsa/src/web/pages/scanconfigs/nvtfamilies.js 90.62% <80.00%> (-2.24%) ⬇️
gsa/src/gmp/models/certbund.js 100.00% <100.00%> (ø)
... and 12 more

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 ecefb99...f8d79d3. Read the comment docs.

@swaterkamp swaterkamp marked this pull request as ready for review May 11, 2021 09:29
@swaterkamp swaterkamp requested a review from a team as a code owner May 11, 2021 09:29
@saberlynx saberlynx merged commit a2503c8 into greenbone:gsa-21.04 May 12, 2021
saberlynx added a commit that referenced this pull request May 12, 2021
Fix changing Trend and Select for NVT-families and whole selection only (backport #2905)
saberlynx added a commit that referenced this pull request May 12, 2021
Fix changing Trend and Select for NVT-families and whole selection only (backport #2905)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
port-to-main Use mergifiy to port PR to master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants