Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bridge] Frontend Development #1908

Open
wants to merge 99 commits into
base: main
Choose a base branch
from

Conversation

Jennievon
Copy link
Contributor

Why this change is needed

Please provide a description and a link to the underlying ticket

https://github.com/ten-protocol/ten-internal/issues/3248

What changes were made as part of this PR

Please provide a high level list of the changes made

Bridge UI development (https://www.figma.com/file/IQKBbkNFftjPqZv0hUGhvX/Spot-the-Ball?type=design&node-id=2084-3347&mode=design&t=20aJUjk5LxlCRSDr-4)

PR checks pre-merging

Please indicate below by ticking the checkbox that you have read and performed the required
PR checks

  • PR checks reviewed and performed

-update health endpoint
- rm nested form
Copy link

vercel bot commented May 30, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
bridge ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 3, 2024 3:05pm

Copy link
Contributor

@0xjba 0xjba left a comment

Choose a reason for hiding this comment

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

LGTM

…nnifer/3248-bridge-develop-bridge-frontend
…nnifer/3248-bridge-develop-bridge-frontend
…nnifer/3248-bridge-develop-bridge-frontend
- add type safety to enhance code reliability
- persist bridge txns to sync storage for better state management
- refactor provider setup to improve configuration and maintenance
- add a status column to bridge txns for clearer txn tracking
- initialize management contract with L1 provider
- add network switching functionality to L1 & gateway redirect for L2
…nnifer/3248-bridge-develop-bridge-frontend
- separate wallet utils and service methods
- set timeout to retry provider detection
Jennievon and others added 13 commits September 3, 2024 18:41
…nnifer/3248-bridge-develop-bridge-frontend
…nnifer/3248-bridge-develop-bridge-frontend
…nnifer/3248-bridge-develop-bridge-frontend
…nnifer/3248-bridge-develop-bridge-frontend
- rename contracts' service file
- refactor `sendNative` fn to support multi-step txn resumption.
- implemented state tracking for each txn step using a resumeStep to ensure txns can be resumed from the last completed step.
- add encrypted storage of pending txns using local storage for data integrity w hashing (wip)
- Integrated logic to save and update txn steps in the pending txns list.
- update the "Finalise" button to be disabled for txns already being processed preventing double submission.
- improve error handling during the multi-step txn resumption process.
…nnifer/3248-bridge-develop-bridge-frontend
…oid infinite loop

- implemented a retry mechanism with a max of 3 retries for "withdrawal already spent" errors during gas estimation.
- add timeout logic to prevent the gas estimation from running indefinitely(returning a default gas limit if necessary)
- improve error handling in gas estimation process.
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.

4 participants