-
Notifications
You must be signed in to change notification settings - Fork 90
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
[frontend] Last fix before release #1083
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1083 +/- ##
============================================
- Coverage 27.11% 26.99% -0.13%
- Complexity 1048 1068 +20
============================================
Files 464 468 +4
Lines 10948 11185 +237
Branches 547 582 +35
============================================
+ Hits 2969 3019 +50
- Misses 7894 8079 +185
- Partials 85 87 +2 ☔ View full report in Codecov by Sentry. |
const handleCheckboxChange = (event) => { | ||
setShowTimeline(event.target.checked); | ||
}; | ||
// const handleCheckboxChange = (event) => { |
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.
Dead code should be removed :)
/> | ||
<span>{t('Show Timeline')}</span> | ||
</div> | ||
{/* <div style={{ marginRight: 10 }}> */} |
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.
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.
LGTM :)
Fix