Closed PeterYurkovich closed 3 months ago
@PeterYurkovich: all tests passed!
Full PR test history. Your PR dashboard.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: PeterYurkovich, zhuje
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Moving the preventQuery and clearError outside of the useEffect or any useMemo and useCallback's prevents the state from being updated correctly, leaving the Korrel8r API to never get called.
Here we are moving the state back into the useEffect since it't not a large enough code migration to break out the useMemo and useCallbacks