feat(example): integrate dragon logs#177
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. 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 comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Bugbot free trial expires on August 9, 2025
Learn more in the Cursor dashboard.
| const SnackBar(content: Text('Logs exported')), | ||
| ); | ||
| }, | ||
| ), |
There was a problem hiding this comment.
Bug: Log Export Button Missing Error Handling
The 'Export Logs' button's onPressed callback lacks error handling for the dragon.DragonLogs.exportLogsToDownload() operation. If the log export fails (throws an exception), the success SnackBar is not displayed, no error feedback is provided to the user, and the unhandled exception can cause the application to crash. A try-catch block is needed to handle failures gracefully and provide appropriate user feedback.
|
Visit the preview URL for this PR (updated for commit 204c953): https://komodo-defi-sdk--pr177-codex-integrate-drag-f2zz6nmh.web.app (expires Tue, 12 Aug 2025 20:05:08 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 7f9f5ac39928f333b6e8fcefb7138575e24ed347 |
* fix(types): comprehensive bip39 validation * fix(market-data-price): try fetch current price from komodo price repository first before cex repository (#167) * fix(market-data-manager): default to KomodoPriceRepository Binance CexRepository supports a limited number of symbols and no longer lists KMD price history after the delisting * refactor(market-data-manager): deduplicate and clarify intent * refactor(review): remove zero check and add log statements also simplify the priceIfKnown function * chore: switch KDF to `dev` Switch KDF to `dev` to allow usage of unreleased changes (particularly related to fees/priorities) * feat: add Flutter Web WASM support with OPFS interop extensions This commit implements comprehensive Flutter Web WASM support by: - Adding opfs_interop.dart with FileSystemDirectoryHandle extensions for values(), keys(), and entries() methods - Implementing JSAsyncIterator type definitions for proper JavaScript interop - Creating WebLogStorageWasm implementation compatible with both JS and WASM targets - Consolidating platform implementations to use unified WASM-compatible storage - Removing redundant WebLogStorage in favor of cross-compatible implementation - Updating platform configuration files for seamless WASM integration The implementation provides backwards compatibility with JavaScript compilation while enabling full WASM support through browser File System APIs that work consistently across both compilation targets. BREAKING CHANGE: WebLogStorage class has been removed in favor of WebLogStorageWasm * Update packages/dragon_logs/lib/src/storage/opfs_interop.dart Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * docs: update repo url * docs: remove repo archive notice * feat(example): integrate dragon_logs (#177) * fix: dart wasm type errors * chore: revert skipped build steps * fix: add GitHub token authentication to prevent API rate limiting Add GitHub authentication token support across all components that make GitHub API requests to resolve rate limiting issues during builds. Changes: - feat(coin-updates): add githubToken parameter to CoinConfigProvider - feat(coin-updates): support GitHub auth in CoinConfigRepository.withDefaults - fix(build-transformer): add debug logging for GitHub API authentication - fix(wallet-cli): auto-use GITHUB_API_PUBLIC_READONLY_TOKEN env variable - style(build-transformer): fix lint issues with cascade operations The CoinConfigProvider was making unauthenticated GitHub API requests, contributing to rate limit errors. All GitHub API clients now properly use the GITHUB_API_PUBLIC_READONLY_TOKEN environment variable when available, increasing rate limits from 60 to 5,000 requests per hour. Fixes: Rate limit exceeded errors during build process * chore(kdf): roll KDF to latest `dev` Roll KDF to the latest `dev` to ensure we are using the latest revision of all unreleased features. * fix: apply cursor fixes --------- Co-authored-by: Francois <takenagain@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Summary
Testing
flutter analyze | tail -n 20flutter test packages/dragon_logs/test(fails: cannot run without a dependency on either "package:flutter_test" or "package:test")https://chatgpt.com/codex/tasks/task_e_68925d6fb2248326b81cf5558d1745cf