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
feat: added layerzero bridge #1267
base: main
Are you sure you want to change the base?
Conversation
Visit the preview URL for this PR (updated for commit 2356fe0): https://astar-apps--pr1267-feat-lz-bridge-e8jiq6to.web.app (expires Thu, 16 May 2024 17:25:24 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: dd76fe72958fe2910fef9d53f0b4539b82b849db |
… into feat/lz-bridge
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, Thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
const abiCoder = new ethers.utils.AbiCoder(); | ||
const byte32Address = abiCoder.encode(['address'], [address]); | ||
|
||
if (byte32Address.length !== 66) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please create a constant with descriptive name
@@ -68,7 +68,8 @@ export default { | |||
blankDestAddress: 'Destination address is blank', | |||
inputtedInvalidAddress: 'Inputted invalid address', | |||
selectedInvalidNetworkInWallet: 'Selected invalid network in your wallet', | |||
balanceNotEnough: 'You do not have enough ETH in your account to pay for the transaction fee', | |||
balanceNotEnough: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This key is already defined under warning
. Also, we have a similar message in error.invalidBalance
. Please reuse.
src/i18n/en-US/index.ts
Outdated
@@ -1035,8 +1036,12 @@ export default { | |||
text: '3rd Party Bridge. Fast and reliable crypto transfers across networks to Astar zkEVM.', | |||
}, | |||
warning32blocks: 'It could take around 10~20mins or more to finalize', | |||
warningLzWithdrawal: | |||
'It could take more than 7 days to finalize the bridge transaction from Astar zkEVM to Astar EVM due to block confirmation fixed by LayerZero', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Time to bridge is reduced to 6 hours
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed, btw, I'm still waiting for DOT to reduce to 6 hours.
Pull Request Summary
Check list
This pull request makes the following changes:
Adds