-
Notifications
You must be signed in to change notification settings - Fork 807
Ransomware table UI #1317
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
Ransomware table UI #1317
Conversation
👉 View analysis in DeepCode’s Dashboard | Configure the bot👉 The DeepCode service and API will be deprecated in August, 2021. Here is the information how to migrate. Thank you for using DeepCode 🙏 ❤️ !If you are using our plugins, you might be interested in their successors: Snyk's JetBrains plugin and Snyk's VS Code plugin. |
e09dc05
to
b408c65
Compare
Codecov Report
@@ Coverage Diff @@
## develop #1317 +/- ##
========================================
Coverage 31.10% 31.10%
========================================
Files 458 458
Lines 13594 13594
========================================
Hits 4229 4229
Misses 9365 9365 Continue to review full report at Codecov.
|
...ey/monkey_island/cc/ui/src/components/report-components/common/RenderFileEncryptionStats.tsx
Outdated
Show resolved
Hide resolved
<p> | ||
This report shows information about the ransomware simulation run by Infection Monkey. | ||
</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove
<p> | ||
This report shows information about the ransomware simulation run by Infection Monkey. | ||
</p> | ||
<div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove
if(successful > 0){ | ||
return (<p className={"text-success"}>{successful} out of {total}</p>); | ||
} else { | ||
return (<p className={"text-danger"}>{successful} out of {total}</p>); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if(successful > 0){ | |
return (<p className={"text-success"}>{successful} out of {total}</p>); | |
} else { | |
return (<p className={"text-danger"}>{successful} out of {total}</p>); | |
} | |
let textClassName = "" | |
if(successful > 0){ | |
textClassName = "text-successful" | |
} else { | |
textClassName = "text-danger" | |
} | |
return (<p className={textClassName}>{successful} out of {total}</p>); |
import React from 'react'; | ||
|
||
|
||
function renderFileEncryptionStats(successful: number, total: number) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can go in the FileEncryptionTable
If some files were encrypted, warning text color should be used. If all files were encrypted, danger text color should be used.
What does this PR do?
Fixes part of #1240.
PR Checklist
Testing Checklist