fix: pre-select LPT as bridge destination token#708
Conversation
The Arbitrum portal defaults the destination token to ETH unless destinationToken is passed explicitly. Add it (using the L1 LPT address, which the portal maps to its L2 counterpart) so the bridge opens with LPT pre-selected on both sides. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe ChangesBRIDGE_LPT_URL query parameter update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related issues
Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This PR fixes the “Bridge LPT” deep link so the Arbitrum bridge UI pre-selects LPT on both the source (Ethereum) and destination (Arbitrum One) sides by explicitly providing destinationToken.
Changes:
- Update
BRIDGE_LPT_URLto includedestinationToken(using the L1 LPT address) so the destination token no longer falls back to ETH.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
The "Bridge LPT" menu option (#705) opened the Arbitrum portal with LPT on the source side but ETH on the destination side, because the portal falls back to ETH when
destinationTokenis absent.Add
destinationTokentoBRIDGE_LPT_URLso LPT is pre-selected on both the Ethereum and Arbitrum One sides.Note: the portal expects the L1 (Ethereum) LPT address for
destinationTokenand resolves the L2 counterpart internally.🤖 Generated with Claude Code
Summary by CodeRabbit