Skip to content

[VIRTS-2188] Change Rule action to a dropdown selection instead of textbox#2157

Merged
clenk merged 4 commits into
masterfrom
emmanvg/virts-2188/make-action-dropdown
May 17, 2021
Merged

[VIRTS-2188] Change Rule action to a dropdown selection instead of textbox#2157
clenk merged 4 commits into
masterfrom
emmanvg/virts-2188/make-action-dropdown

Conversation

@emmanvg
Copy link
Copy Markdown
Contributor

@emmanvg emmanvg commented May 17, 2021

Description

Make "action:" under the rules modal a dropdown with the only valid options. In the past this would be a textbox that would allow any value, but in reality there are only two valid values ALLOW or DENY. Hence, it makes sense to simplify this form.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Go fact sources, click view sources, click add rule. Type a trait, a match and select a value from the dropdown modal ALLOW or DENY. Check under data/sources/UUID.yml that an entry exists with the proper ALLOW or DENY value. Re-open fact sources, load the test fact source and view the rules, check that nothing has changed.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works

@codecov
Copy link
Copy Markdown

codecov Bot commented May 17, 2021

Codecov Report

Merging #2157 (b318593) into master (394852a) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2157   +/-   ##
=======================================
  Coverage   66.43%   66.43%           
=======================================
  Files          70       70           
  Lines        5225     5225           
=======================================
  Hits         3471     3471           
  Misses       1754     1754           

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 394852a...b318593. Read the comment docs.

Copy link
Copy Markdown
Contributor

@clenk clenk left a comment

Choose a reason for hiding this comment

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

LGTM; this has the exact same changes as #2144 which was approved by me and @b17zr.

Copy link
Copy Markdown
Contributor

@mrengstrom mrengstrom 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 to me

@clenk clenk merged commit 515b4ba into master May 17, 2021
@clenk clenk deleted the emmanvg/virts-2188/make-action-dropdown branch May 17, 2021 14:33
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