chore(extension): migrate extension branch from private repository #2513
+1,270
−233
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
remove cloudflare workflows
* Fix electrum coin activation on native platforms Added conditional checks based on the current platform. WSS is not supported on native platforms, but required for Web. * Rename `_removeElectrumsWithoutWSS` * make constructor const --------- Co-authored-by: smk762 <[email protected]> Co-authored-by: Charl (Nitride) <[email protected]>
Fiat page inputs cleanup
Fix coin faucet button
* Upgrade to Flutter 3.19.6 Update `flutter_slidable` to 3.1.0 * Migrate to Gradle's declarative plugins block Upgrade to gradle build tools 8.7 Set build target to API version 34 * Update pinned Flutter version in docs * Add docs for serving localhost through https Serving through HTTPS is sometimes necessary due to security policies (i.e. Bitrefill). This functionality was only merged in Flutter 3.19 with PR [#60704](flutter/flutter#60704)
Fix password confirm validation issue so that the field auto-validates for all inputs after the user’s first submission attempt.
* Transfer from archive repository as new package Copied from `add/update-coin-config` branch with PR #1336 in `komodo-wallet-archive` * Add coins.json models * Add Hive box persistence provider and interface Add as a new package to allow for use in portfolio graphs * Make bulk operation implementations compulsory * Replace KeyValueDatabase with PersistenceProvider API * Make Coin model fields nullable * Add coins config models and adapters * Refactor Hive adapters into a separate folder * Add coin commit hash storage * Change adapter fields to nullable and remove `required` annotations * Add Lazy box implementation * Combine coins and coins_config into a single box and model * Allow exceptions to move up Don't handle exceptions in the package * Add documentation with usage examples * Add documentation comments to the data layer classes * Add db/table exists check to persistence layer * Add interval refresh state and events and consolidate repository * Add periodic update event and handlers * Fix linter warnings * Save the bundled coin configs to Hive if no box exists * Add new coins repository to the existing repository Interim solution * Revert coins repo field accessibility to private * Add local config exists check and add documentation * Add fallback for missing local asset files * Move runtime coin updates into a Isolate with `compute` * Fix missing coins not activating on refresh Use the coin repository to get known coins instead of the parser, so that the coins saved to Hive can be passed to the API * Revert breaking change to `CoinsBloc` This was introduced to fix a bug during development that is now fixed by changes to the `CoinsRepo.getKnownCoins` and mm2 api initialisation functions. * Change to exception-based error handling * Add log statements for errors during raw rx send * Fix incorrect coins format being sent to mm2 Required fields from `coins.json` were missing from the `Coin` class when mapping to JSON. Add and use a new function explicitly for the `coins.json` format required on startup. * Fix flutter analyze warning and missing coin configs * Change `int` and `double` to `num` to fix parsing errors * Fix Hive initialization in Isolates on native platforms * Move coin and electrum removal from package to app bloc * Fix duplicate type adapter registration Only re-initialize Hive on native platforms where isolates run in separate processes. * Remove coins without WSS electrums from storage on web * Fix flutter analyze warning * Read runtime coin updates branch and url from config file Move `coins_ci.json` to assets root and update fields. Use the config to populate the default values for the GitHub provider . * Update `fetch_coins.py` script to read from runtime updates config * Create required directory in assets folder * Change to connection reference and non-null parameter
Fiat page uses simple spinner loading
Fix password confirmation error state not automatically updating
Implement Dart build script for fetching API binaries at build-time.
* Add dart fetch coin assets script * Update project setup documentation * Keep coin assets directories in folder structure * Remove references to `fetch_coins.py` from GitHub Actions scripts
* SEO updates and some optimizations * make workflow_dispatch work * make workflow_dispatch work * removes preload of main.dart.js as it wasn't preventing sw from making another request * adds robots file, better loading image * updates generated html * fixes komodoplatform logo url, makes the app use canvas.wasm from our host and make it be cached by service worker * make fallback font(roboto) cached by service worker * removes unnecessary preconnect * reverts release mode to --profile * sets prefer_related_applications to false * replaces loader icon with higher res, lower size image * update favicons and other icons * updates buit index.html
* Init Defi API installer package build_time_defi_installer init using the command `dart create -t cli build_time_defi_installer` * Rename Defi installer to general build transformer Rename Defi installer to general build transformer to reflect the expanded scope of the transformer’s responsibilities. * Relocate build steps Relocate build steps authored by @takenagain and @naezith in preparation for their integration into the build system. * Further renaming * Create abstract build step interface * Adapt defi API step to new structure * Revert removed coin asset fetching code Credit to @takenagain for the changes * Remove renamed orphaned pubspec file * Remove redundant app build setup * Update build transformer pubspec description * Add CLI entry for defi fetcher * Remove `index.html` Remove the committed `index.html` because it is automatically generated as part of the build process. * add `intex.html` to gitignore add `intex.html` to gitignore since it is automatically generated in the build process as part of the recent build-time changes * Name build steps according to the convention * Add step for copying platform assets * Remove more redundant build scripts & update docs * Implement the build step manager Implement the build step manager that manages execution of build steps * Fix broken coins runtime update path * Update outdated package * Revert `.gitignore` * Ignore `index.html` to build at build-time * Revert "Ignore `index.html` to build at build-time" This reverts commit 4cefc48b21853a2e36030bc7296acd3f836475cb. * Fix broken GH workflows * Relax lint rule * Improvements and further workflow fixes * Update GH Actions * More GH Actions updates * Update bundled commit hash on run and update save method * Refactoring and improvements * Fix failing test * Update build config coins commit * GH Workflow fix * Workflow test #2 * Workflow test #3 * Fix asset bundle path * Revert "Workflow test #3" This reverts commit 7b137ca6ec27c3046c446aabc190ad32a01a018d. * GH Workflows test nr 4 --------- Co-authored-by: Francois <[email protected]>
CI build system fixes & add functionality to skip the API update build step if `should_update` in the build config is `false` (previously this value was ignored).
Fix bootstrap loader null exception encountered when doing hot restart in debug mode development.
Fix missing CI steps
0.7.3
Fix Enter submit on forms not working
This reverts commit a623444ea75e4d074419bcc9689585ca819beceb.
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
takenagain
force-pushed
the
merge/extension-private-public
branch
from
f9c03f2 to
ef7b4b2
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.