-
Notifications
You must be signed in to change notification settings - Fork 240
feat(staking): add comprehensive staking and Flutter wasm overhaul #3052
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
|
Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing Touches🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
|
Visit the preview URL for this PR (updated for commit 87db369): https://walletrc--pull-3052-merge-dgr9qeya.web.app (expires Mon, 18 Aug 2025 10:34:44 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: f66a4ff03faa546f12f0ae5a841bd9eff2714dcc |
Replace deprecated `hive_ce` with community fork for improved performance and Flutter web wasm support
…into add/staking # Conflicts: # .docker/build.sh
…into add/staking # Conflicts: # lib/bloc/app_bloc_root.dart
Shotgun-precache images for SDK on mobile platforms
Use SDK’s robust JSON extension methods to resolve strict type casting on Flutter wasm web.
Tested with the wasm build flag to ensure that the looser requirements are still WASM compatible
Banxa does not include the required cross-origin headers in their responses, so Wasm cross-origin isolation requirements cause the page to fail to load
…into add/staking # Conflicts: # lib/views/wallet/wallet_page/common/expandable_coin_list_item.dart # lib/views/wallets_manager/widgets/wallet_simple_import.dart # packages/komodo_ui_kit/pubspec.lock # pubspec.lock
| _currentPrecacheOperation = Completer<void>(); | ||
|
|
||
| // Load all icons similaneously if user is not on mobile device | ||
| final shouldShotgunLoad = !Platform.isAndroid && !Platform.isIOS; |
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.
| ColoredBox(color: Colors.green), | ||
|
|
||
| // _buildDelegationsTab(), | ||
| ], |
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.
Bug: Staking Page Tabs Display Incorrect Content
The Rewards and Delegations tabs in StakingPage's TabBarView are showing placeholder ColoredBox widgets instead of their intended content. This prevents users from accessing the staking rewards and delegations functionality, likely due to debugging code being accidentally committed.
Summary
This PR adds comprehensive staking functionality to Komodo Wallet, enabling users to stake Cosmos-based assets and earn rewards through delegation.
What's Added
Core Functionality
Technical Implementation
Architecture
UI Components
Integration
Technical Details
Files Added
Key Features
Testing
How to Test
Test Scenarios
Notes
Breaking Changes
None - this is a purely additive feature.
Dependencies
Statistics: 48 files changed, 4,109 insertions(+), 465 deletions(-)
Ready for: Code review, testing in debug mode