-
Notifications
You must be signed in to change notification settings - Fork 786
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 quickstart - Remove "Congrats" message and change header #1314
Ransomware quickstart - Remove "Congrats" message and change header #1314
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #1314 +/- ##
===========================================
+ Coverage 30.87% 31.10% +0.22%
===========================================
Files 454 458 +4
Lines 13545 13596 +51
===========================================
+ Hits 4182 4229 +47
- Misses 9363 9367 +4
Continue to review full report at Codecov.
|
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 the bottom margin of the title and change "Get Started" to "Getting Started". Also, add 20px of space between buttons and next section in the page
monkey/monkey_island/cc/ui/src/components/pages/RunServerPage.js
Outdated
Show resolved
Hide resolved
monkey/monkey_island/cc/ui/src/components/pages/RunServerPage.js
Outdated
Show resolved
Hide resolved
monkey/monkey_island/cc/ui/src/components/pages/RunServerPage.js
Outdated
Show resolved
Hide resolved
monkey/monkey_island/cc/ui/src/components/pages/RunServerPage.js
Outdated
Show resolved
Hide resolved
09731f6
to
a5151a6
Compare
What does this PR do?
Fixes "Remove" task from #1241
PR Checklist
Was the CHANGELOG.md updated to reflect the changes?Was the documentation framework updated to reflect the changes?Testing Checklist
Added relevant unit tests?Explain Changes
Are the commit messages enough? If not, elaborate.