Fix transfer completion bug in manual transfer claims (#2651)

* Fix transfer completion bug in manual transfer claims

When claiming a manual transfer, the useTrackTransfer hook sometimes marked the transfer as complete before the async function triggered by the claim button finished. This allowed users to start a new transfer prematurely, causing the new transfer to be erroneously marked as complete. We should rely solely on the useTrackTransfer hook to set the transfer as complete and handle the redeem transaction.

* move event back

wormhole-connect/src/views/v2/Redeem/index.tsx

@@ -26,7 +26,6 @@ import { RouteContext } from 'contexts/RouteContext';
 import useTrackTransfer from 'hooks/useTrackTransfer';
 import PoweredByIcon from 'icons/PoweredBy';
 import { SDKv2Signer } from 'routes/sdkv2/signer';
-import { setRedeemTx, setTransferComplete } from 'store/redeem';
 import { setRoute } from 'store/router';
 import { displayAddress, millisToHumanString } from 'utils';
 import { interpretTransferError } from 'utils/errors';
@@ -483,13 +482,6 @@ const Redeem = () => {
  console.error(e);
  }
  if (txId !== undefined) {
- dispatch(setRedeemTx(txId));
-
- // Transfer may require an additional step if this is a finalizable route
- if (!routes.isFinalizable(route)) {
- dispatch(setTransferComplete(true));
- }
-
  config.triggerEvent({
  type: 'transfer.redeem.success',
  details: transferDetails,