Skip to content

Improve information about bridge state#371

Open
CedrikNikita wants to merge 2 commits intodevelopfrom
feature/improve-information-about-bridge-state
Open

Improve information about bridge state#371
CedrikNikita wants to merge 2 commits intodevelopfrom
feature/improve-information-about-bridge-state

Conversation

@CedrikNikita
Copy link
Collaborator

@CedrikNikita CedrikNikita commented Nov 20, 2025

part of #361


Note

Introduce step-aware bridge flow (allowance vs bridge), pending/confirmed activity updates, user/network timeouts, and expanded i18n/error messages.

  • Bridge UI/Logic (src/features/ae-eth-bridge/components/AeEthBridge.tsx):
    • Add bridgeStep state to distinguish allowance vs. bridge phases; surface status in UI and button text.
    • Improve UX with user/network timeout hints during confirmations and long waits; clearer confirming messages.
    • Track recent activities: add pending bridge activity on submit and update status (confirmed/failed) via updateActivityStatus.
    • Enhance error handling with specific, localized messages (insufficient funds, user rejection, network/timeout, gas, predictable failure); normalize catch blocks and cleanup state.
    • Minor: remove unused token display helper; refine effect/handler deps; ensure balance refetch after bridge.
  • Locales (src/locales/en/dex.json):
    • Add/rename keys for transaction status labels, explorer labels, and long-wait hints (e.g., waitingForUserOrNetwork, directionLabel, etc.).
    • Localize dialog labels and various bridge messages (approval/allowance/processing/confirmation).

Written by Cursor Bugbot for commit 0f2046f. This will update automatically on new commits. Configure here.

@netlify
Copy link

netlify bot commented Nov 20, 2025

Deploy Preview for fancy-gelato-7cdad5 ready!

Name Link
🔨 Latest commit 0f2046f
🔍 Latest deploy log https://app.netlify.com/projects/fancy-gelato-7cdad5/deploys/691f6186e4d02e00076ba54b
😎 Deploy Preview https://deploy-preview-371--fancy-gelato-7cdad5.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@CedrikNikita CedrikNikita force-pushed the feature/improve-information-about-bridge-state branch from 99739a9 to 2ab2d21 Compare November 20, 2025 18:17
@CedrikNikita CedrikNikita force-pushed the feature/improve-information-about-bridge-state branch from 2ab2d21 to 0f2046f Compare November 20, 2025 18:44

setButtonBusy(false);
}, [asset, destination, normalizedAmount, isValidDestination, aeternityAddress, direction, sdk]);
}, [asset, destination, normalizedAmount, isValidDestination, aeternityAddress, direction, sdk, aeBalances, ethBalances, t]);
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Missing balance refresh after bridge transaction

The bridgeToEvm function is missing a call to refetchBalances() after the transaction attempt (success or failure). This prevents the UI from automatically updating the user's token balances to reflect gas usage and the bridged amount, leaving outdated information displayed.

Fix in Cursor Fix in Web

@Andreea167
Copy link

Andreea167 commented Nov 21, 2025

I executed a bridge from ETH -> aeETH first time and went well -> took indeed 2-3 mins and it was executed

Then I tried to make a second transaction, again from ETH->aeETH -> I confirm it in the wallet and then the message stayed like that since 10-15 minutes without effect, and without be some "pending" message in the wallet

So seems that, at least for the second time of performing, I get this "stuck page"

Screenshot 2025-11-21 at 09 50 16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants