yardz opened a new pull request #13220:
URL: https://github.com/apache/superset/pull/13220


   ### SUMMARY
   - Creation of a `componentDidMount` hook. The reason for this hook is so 
that there are fewer lint errors and there is no need to put ``// 
eslint-disable-next-line react-hooks/exhaustive-deps`` throughout the project
   - Updated the ``useEffect(effect, [])`` to use the new hook (remove some 
lint errors and improve reading)
   - Switched FlashProvider to function component and created tests for it 
(Also simplify the props he receives)
   - I separated the creation of the Store from the App file. To facilitate the 
maintenance and creation of new reducers. And also to facilitate the testing of 
some components that need to perform dispatches.
   
   Despite the change in the FlashProvider interface, no change in behavior 
should occur in the application.
   
   ### TEST PLAN
   All tests must pass


----------------------------------------------------------------
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]

Reply via email to