Improve logic, add new swap view #24
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull Request Overview
This PR improves the logic and adds a new swap view to replace the existing home view implementation. The changes consolidate staking, minting, burning, boosting, and other vault operations into a unified SwapView interface.
Key changes:
- Refactored context and state management for better separation of concerns
- Added new components for input/output handling and transaction management
- Improved type safety and removed deprecated functionality
Reviewed Changes
Copilot reviewed 261 out of 402 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/views/SwapView/ | New unified view for all vault operations (stake, mint, burn, boost, etc.) |
| src/store/store/vault/ | Simplified state structure by removing unused slices |
| src/views/HomeView/ | Removed old home view implementation |
| src/layouts/modals/ | Updated modal imports and transaction flow handling |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| return () => { | ||
| unboost.percentField.reset() | ||
| } | ||
| }, []) |
There was a problem hiding this comment.
The useEffect cleanup function is missing the dependency array for unboost.percentField.reset. Add unboost.percentField.reset to the dependency array to ensure the effect updates if the reset function changes.
| }, []) | |
| }, [unboost.percentField.reset]) |
No description provided.