-
Notifications
You must be signed in to change notification settings - Fork 198
fix: remove WETH/FOX from rFOX stake modal asset selection #11780
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
Conversation
The Arbitrum WETH/FOX Rewards program has been sunset, so WETH/FOX should no longer be selectable as a staking asset in the Stake modal. Users can still unstake and claim their existing positions. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📝 WalkthroughWalkthroughModified StakeInput component to exclude the Arbitrum WETH/FOX staking asset from the user-selectable list. The change filters out the specific asset ID and appends the L1 asset ID, preventing WETH/FOX selection while maintaining L1 staking availability. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@src/pages/RFOX/components/Stake/StakeInput.tsx`:
- Around line 96-102: The useMemo for computing stakingAssetIds references
supportedStakingAssetIds and uniV2EthFoxArbitrumAssetId but they are missing
from the dependency array; update the dependency array of the useMemo that
defines stakingAssetIds to include l1AssetId, supportedStakingAssetIds, and
uniV2EthFoxArbitrumAssetId so the memo correctly tracks all used values (keep
the logic that filters out uniV2EthFoxArbitrumAssetId and concatenates l1AssetId
unchanged).
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Description
Removes WETH/FOX from the staking asset selection dropdown in the Stake modal since the Arbitrum WETH/FOX Rewards program has been sunset. Users can still unstake and claim existing positions.
Issue (if applicable)
closes #11779
Risk
Low - UI-only change that filters out a sunset staking option from selection.
None. This only affects the asset selection UI in the rFOX Stake modal.
Testing
Engineering
Operations
Screenshots (if applicable)
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.