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
refactor : Transform FilterScopeModal to Typescript #11964
Conversation
Codecov Report
@@ Coverage Diff @@
## master #11964 +/- ##
==========================================
- Coverage 63.75% 63.65% -0.11%
==========================================
Files 941 941
Lines 45654 45709 +55
Branches 4389 4396 +7
==========================================
- Hits 29106 29094 -12
- Misses 16371 16436 +65
- Partials 177 179 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
super(props); | ||
|
||
this.modal = React.createRef(); | ||
this.handleCloseModal = this.handleCloseModal.bind(this); | ||
} | ||
|
||
handleCloseModal() { | ||
handleCloseModal(): any { |
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.
handleCloseModal(): any { | |
handleCloseModal(): void { |
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.
Just one nit (easy tweak, unless I'm missing something), and I think this is good to go!
SUMMARY
Transform FilterScopeModal to Typescript