fix: rearranged the order of statements inside the cleanup function

[BiswaViraj]

What change does this PR introduce?

Why was this change needed?

inside use-socket-controller.ts useEffect

return () => {
     socket = null;
     socket?.disconnect();
};

the order of statements were wrong
socket.disconnect should be invoked before setting it to null

Other information (Screenshots)

[linear]

NV-1555 websocket sends 2 disconnection requests on reload