tanmaylaud commented on pull request #11002: URL: https://github.com/apache/incubator-superset/pull/11002#issuecomment-696915485
> @tanmaylaud I'm not sure "functional components is recommended" is really a statement that can be broadly applied to all existing code. Functional components + hooks work significantly different than class components, so there is a non-trivial refactor needed to reimplement the existing logic using hooks. As with all substantial refactors, substantial testing is required to ensure functionality has not changed. I see you removed some tests in that PR. Looks like a test case that verifies the timer is reset is missing. I am not denying that there is some issue with the current implementation but I am only asking that we forward fix the issue. The test cases were changed(not removed) . I didn't do it randomly. There are limitations on how the useEffect hooks can be tested. It would be great if you can let me know the issue in the current code. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
