-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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: Add reset button for settings and use settings modalv2 for all swap & liquidity #11291
base: develop
Are you sure you want to change the base?
feat: Add reset button for settings and use settings modalv2 for all swap & liquidity #11291
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
|
f210791
to
1679ff0
Compare
9608500
to
ef82e75
Compare
a81d0c8
to
0312002
Compare
Remove liquidity currently using old settings which includes obsolote fast routing setting
To reproduce
Go to remove lp (v2 / stable)
Click slippage tolerance
See old settings modal
PR-Codex overview
This PR focuses on enhancing the user experience by updating various components and hooks related to settings, modals, and user preferences, particularly around slippage, notifications, and token risk. It transitions to a new modal system and optimizes user preference handling.
Detailed summary
SettingsModal
to useSettingsModalV2
with improved state management.ModalV2
for better modal handling across components.useAllowNotifications
hook to return default values.