-
Notifications
You must be signed in to change notification settings - Fork 240
chore(release): v0.9.0 - HD support, Unified codebase, Custom tokens, and SDK migration #2526
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
Merged
Changes from 103 commits
Commits
Show all changes
104 commits
Select commit
Hold shift + click to select a range
337efc0
use gha runners
smk762 cf5849c
fix(tests): failing unit and integration tests (#167)
takenagain 3a29513
add arbitrum coin type (#166)
naezith 2a0e9b3
Enable Test Coins Setting (#165)
naezith 70907c0
chore(deps): Upgrade deps/Flutter and add support for native mobile b…
takenagain 654530e
DEX Swap form gets filled with URL parameters (#162)
naezith 210aecc
fix onPopPage deprecated (#172)
naezith e215610
add are you sure prompt at close/refresh (#173)
naezith 4b9d31d
fix(log-export): cross platform log export (#174)
takenagain 8dff320
feat(fiat-on-ramp): cross-platform fiat on-ramp (#170)
takenagain 56ec51d
Private Key Export (#183)
naezith c9f6acc
Hot-fix for CDN fallback of missing coin icons (#180)
CharlVS 5448183
fix(lp-tools): ux inconsistency and coin amounts while order is being…
takenagain 1aa5788
feat(system-time-check): add fallback world time apis (#182)
takenagain bbac902
feature: kdf sdk integration part 1 - startup (#177)
takenagain 5524c62
bump sdk version (#190)
takenagain 708e165
update kdf hashes in build_config (#189)
smk762 01bed5c
Merge coin icon fallback
CharlVS fb0c048
Fix zero balance issue of tokens without parent coin gas (#186)
naezith 62b4b05
fix(trezor): breaking hd wallet balance status rpc (#194)
takenagain 9a18f5b
limit max_connected to 1 for electrum activation (#195)
takenagain 8d1d85e
Disable trading features
CharlVS f387a96
bug: Fix incorrect wallet mode logic
CharlVS 9f46ca5
feat: Hide incorrect time banner in wallet-only mode
CharlVS 7fe4ee4
Bug: Fix missing coin detail buttons in wallet-only mode
CharlVS 9512dd2
fix: missing conditional wallet mode check
CharlVS 9eb8fed
SDK Integration & HD Coin Details page + Coin Addresses (#178)
naezith eda3b2a
hide trading bot setting
naezith 9b66614
default to wallet route parser if wallet only
naezith c6f04b8
Merge branch 'main' into merge/dev-hotfixes-v0_8_2
takenagain eb912ae
add removed max maker vol models
takenagain 853517b
fix merge conflicts with moved/modified code
takenagain 4e7f329
wallet only logout confirmation description
naezith 6de0759
Merge branch 'dev' into disable-trading-bot-setting
naezith 7a3102e
chore(sdk): Update KDF for custom seed import fix (#2511)
CharlVS 3b40883
HD Address Management & Seed Import (#2510)
naezith 73ac363
feat(hd): HD withdrawals + breaking SDK changes (#2520)
CharlVS 869c8fa
fix(router): frozen layout on navigation and resize (#2521)
takenagain 56bcaf6
Merge branch 'main' into merge/main-to-dev
takenagain 6f1baef
perf(add-coins): improve "Add Assets" coin list loading speed (#2522)
takenagain 9a7c1da
Merge pull request #2524 from KomodoPlatform/merge/main-to-dev
CharlVS 3176b11
fix(transaction-history): incorrect transaction history shown when sw…
takenagain 3b24f4b
fix(coins-bloc): deactivated coins not triggering bloc rebuilds (#2527)
takenagain 538aadf
feat: Custom Token Import UI and BLoC (#2515)
takenagain b11782a
chore(flutter): Upgrade to Flutter 3.29 and Lock Dependencies for Rel…
CharlVS e9abd03
feat(hd): HD withdrawals and portfolio overview (#2530)
CharlVS 6b06f15
ci: upgrade workflows to flutter 3.29 (#2531)
takenagain c4b67e8
chore(deps): update SDK commit hash (#2532)
takenagain 8fa543a
fix: missing segwit badges (#2545)
takenagain eb01662
build: fix Docker and Dev Container Flutter 3.29 build error (#2542)
takenagain 92cb33e
docs: update the project and flutter setup docs (#2534)
takenagain 75c9e40
bug(auth): Remove `bip39` dependency and fix registration flow on Win…
CharlVS 073df90
bug(registration): Fix incorrect function signature (#2551)
CharlVS 1b01b89
chore(sdk): Upgrade SDK version (#2552)
CharlVS d5027d9
Fix some grammar in the EULA/ToC (#2540)
smk762 21b82ba
ci: add workflows to run builds for mobile and desktop platforms (#2528)
takenagain 5e70115
chore(sdk): Roll KDF SDK to `af79b7d09ab3f3b45ff6767b9556c514f09b25ce…
CharlVS b5f2e5c
fix(auth): backup seed notification banner not appearing for newly cr…
takenagain 9ce4ef2
chore(sdk): Add coin deprecation notices (#2557)
CharlVS 746a245
fix(auth): missing login failed error and BIP39 validation clarificat…
takenagain d363ae2
chore: remove build_config.json (#2550)
takenagain a596bc6
fix(market-metrics): hd mode switching values shown, graph interactio…
takenagain ab0c3b3
refactor: migrate to SDK balance manager and integrate ThemeExtension
CharlVS e96767f
fix(coins-repo): remove price refresh timer
takenagain db6e004
fix(coins-repo): remove enabled assets broadcast on balance update
takenagain f83882d
fix: coin balance used in place of coin usd balance
takenagain 32bef27
bug(ui): Fix “Receive” button
CharlVS aec8cac
chore(cex): Delete cex market data package migrated to SDK
CharlVS 6082016
chore(sdk): Roll SDK version
CharlVS d30244a
fix(prices): add change percentage over 24 hour period to cache
takenagain 064ffc9
fix(dex): available balance not updating
takenagain c1282de
feat(feedback): implement BetterFeedback and Trello integration
CharlVS d513ddc
bug(ci): Fix Trello secrets not being passed to Flutter build
CharlVS 08b081c
chore(sdk): Role SDK (#2585)
CharlVS aa70cae
feat: Rework unauthenticated assets list (#2579)
CharlVS 82392a7
fix(coins-bloc): disabled coins reactivating in suspended coin timer …
takenagain 5fcc142
chore(release): Upgrade dependencies for RC security review (#2589)
CharlVS 5ef9d2e
Add multi-address support to "Faucet" button (#2533)
TazzyMeister f600dfc
feat(ui/ci): Add feedback provider for internal feedback service (#2586)
CharlVS 71ed192
build(ios,macos): update pod file locks to match pubspec (#2594)
takenagain e62e5cd
Update fiat_payment_method_card.dart (#2588)
TazzyMeister 70827b6
chore(deps): upgrade SDK commit hash (#2597)
takenagain 73a3e04
fix(custom-token-import): breaking asset constructor change (#2598)
takenagain fd487aa
chore(deps): upgrade major and minor dependency versions (#2602)
takenagain b98d23d
chore(deps): roll SDK version (#2603)
takenagain 9de086c
feat(auth): SDK password update migration (#2580)
CharlVS f3597ab
fix(withdraw-form): validation, dropdown icon, and maximum amount fie…
takenagain db00163
Chore/localise hardcoded strings (#2587)
TazzyMeister 200cdf7
Aligned Column Headers (#2577)
TazzyMeister 1fa554f
fix: spelling mistake
CharlVS a591c1c
chore: roll SDK packages (#2606)
CharlVS 04696c9
fix(nft): add nft v2 and hd wallet support (#2566)
takenagain 9b46ffe
feat(fiat-onramp): hd wallet address selection support (#2570)
takenagain aaf19e4
docs: Add SDK roll PR image
CharlVS 5a6ad28
ci(sdk): Create script/action for SDK auto roll (#2615)
CharlVS 47300e1
fix(system-health-check): add additional time providers and update UR…
takenagain a4bc9a7
fix(dex): crash on buy coin dropdown selection (#2624)
takenagain 7335ab5
fix(fiat-onramp): banxa onramp flow erroneous failure popup and retur…
takenagain acd39b6
ci(security): add osv scanner workflow (#2656)
CharlVS bf6ad80
Meta title desc updated (#2645)
gcharang 9830a0f
fix(sdk): Apply SDK fix for coins bug and KDF release update (#2637)
CharlVS 713d156
feat: expand analytics events (#2651)
CharlVS a83fd14
chore: release notes for v0.9.0 (#2655)
CharlVS c0cf652
fix(auth): wallet password validation in creation and update flows (#…
takenagain File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,117 @@ | ||
| FROM docker.io/ubuntu:22.04 | ||
|
|
||
| ENV FLUTTER_VERSION="3.29.2" | ||
| ENV USER="komodo" | ||
| ENV USER_ID=1000 | ||
| ENV PATH=$PATH:/opt/flutter/bin | ||
| ENV PATH=$PATH:/android-ndk/bin | ||
| ENV ANDROID_HOME=/opt/android-sdk-linux \ | ||
| LANG=en_US.UTF-8 \ | ||
| LC_ALL=en_US.UTF-8 \ | ||
| LANGUAGE=en_US:en | ||
| ENV TMPDIR=/tmp/ \ | ||
| ANDROID_DATA=/ \ | ||
| ANDROID_DNS_MODE=local \ | ||
| ANDROID_ROOT=/system | ||
|
|
||
| ENV ANDROID_SDK_ROOT=$ANDROID_HOME \ | ||
| PATH=${PATH}:${ANDROID_HOME}/cmdline-tools/latest/bin:${ANDROID_HOME}/platform-tools:${ANDROID_HOME}/emulator | ||
|
|
||
| # comes from https://developer.android.com/studio/#command-tools | ||
| ENV ANDROID_SDK_TOOLS_VERSION=11076708 | ||
|
|
||
| # https://developer.android.com/studio/releases/build-tools | ||
| ENV ANDROID_PLATFORM_VERSION=35 | ||
| ENV ANDROID_BUILD_TOOLS_VERSION=35.0.1 | ||
|
|
||
| # https://developer.android.com/ndk/downloads | ||
| ENV ANDROID_NDK_VERSION=27.2.12479018 | ||
|
|
||
| RUN apt update && apt install -y sudo && \ | ||
| useradd -u $USER_ID -m $USER && \ | ||
| usermod -aG sudo $USER && \ | ||
| echo "$USER ALL=(ALL) NOPASSWD: ALL" >> /etc/sudoers && \ | ||
| mkdir -p /workspaces && \ | ||
| chown -R $USER:$USER /workspaces && \ | ||
| chown -R $USER:$USER /opt | ||
|
|
||
| RUN apt-get update -y && \ | ||
| apt-get install -y --no-install-recommends \ | ||
| ca-certificates \ | ||
| build-essential \ | ||
| libssl-dev \ | ||
| cmake \ | ||
| llvm-dev \ | ||
| libclang-dev \ | ||
| lld \ | ||
| gcc \ | ||
| libc6-dev \ | ||
| jq \ | ||
| make \ | ||
| pkg-config \ | ||
| git \ | ||
| automake \ | ||
| libtool \ | ||
| m4 \ | ||
| autoconf \ | ||
| make \ | ||
| file \ | ||
| curl \ | ||
| wget \ | ||
| gnupg \ | ||
| software-properties-common \ | ||
| lsb-release \ | ||
| libudev-dev \ | ||
| zip unzip \ | ||
| nodejs npm \ | ||
| binutils && \ | ||
| apt-get clean | ||
|
|
||
| USER $USER | ||
|
|
||
| RUN set -e -o xtrace \ | ||
| && cd /opt \ | ||
| && sudo chown -R $USER:$USER /opt \ | ||
| && sudo apt-get update \ | ||
| && sudo apt-get install -y jq \ | ||
| openjdk-17-jdk \ | ||
| # For Linux build | ||
| clang cmake git \ | ||
| ninja-build pkg-config \ | ||
| libgtk-3-dev liblzma-dev \ | ||
| libstdc++-12-dev \ | ||
| xz-utils \ | ||
| wget zip unzip git openssh-client curl bc software-properties-common build-essential \ | ||
| ruby-full ruby-bundler libstdc++6 libpulse0 libglu1-mesa locales lcov \ | ||
| libsqlite3-dev --no-install-recommends \ | ||
| # for x86 emulators | ||
| libxtst6 libnss3-dev libnspr4 libxss1 libatk-bridge2.0-0 libgtk-3-0 libgdk-pixbuf2.0-0 \ | ||
| && sudo rm -rf /var/lib/apt/lists/* \ | ||
| && sudo sh -c 'echo "en_US.UTF-8 UTF-8" > /etc/locale.gen' \ | ||
| && sudo locale-gen \ | ||
| && sudo update-locale LANG=en_US.UTF-8 \ | ||
| && wget -q https://dl.google.com/android/repository/commandlinetools-linux-${ANDROID_SDK_TOOLS_VERSION}_latest.zip -O android-sdk-tools.zip \ | ||
| && mkdir -p ${ANDROID_HOME}/cmdline-tools/ \ | ||
| && unzip -q android-sdk-tools.zip -d ${ANDROID_HOME}/cmdline-tools/ \ | ||
| && mv ${ANDROID_HOME}/cmdline-tools/cmdline-tools ${ANDROID_HOME}/cmdline-tools/latest \ | ||
| && sudo chown -R $USER:$USER $ANDROID_HOME \ | ||
| && rm android-sdk-tools.zip \ | ||
| && yes | sdkmanager --licenses \ | ||
| && sdkmanager platform-tools \ | ||
| && git config --global user.email "[email protected]" \ | ||
| && git config --global user.name "Komodo Platform" \ | ||
| && yes | sdkmanager \ | ||
| "platforms;android-$ANDROID_PLATFORM_VERSION" \ | ||
| "build-tools;$ANDROID_BUILD_TOOLS_VERSION" | ||
|
|
||
| RUN cd /opt && \ | ||
| curl -O https://storage.googleapis.com/flutter_infra_release/releases/stable/linux/flutter_linux_${FLUTTER_VERSION}-stable.tar.xz && \ | ||
| tar -xvf flutter_linux_${FLUTTER_VERSION}-stable.tar.xz -C /opt && \ | ||
| rm flutter_linux_${FLUTTER_VERSION}-stable.tar.xz && \ | ||
| flutter config --no-analytics && \ | ||
| flutter precache && \ | ||
| yes "y" | flutter doctor --android-licenses && \ | ||
| flutter doctor && \ | ||
| flutter update-packages --verify-only && \ | ||
| mkdir -p /workspaces/komodo-wallet && \ | ||
| chown -R $USER_ID:$USER_ID /workspaces/komodo-wallet | ||
This file was deleted.
Oops, something went wrong.
This file contains hidden or 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
This file was deleted.
Oops, something went wrong.
This file contains hidden or 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
Oops, something went wrong.
Oops, something went wrong.
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.
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.
🛠️ Refactor suggestion
Use of Sudo in RUN Command.
The RUN command leverages
sudomultiple times. According to static analysis (Hadolint DL3004), usingsudocan lead to unpredictable behavior in Docker images. Consider using a tool likegosufor controlled privilege escalation within the container.🧰 Tools
🪛 Hadolint (2.12.0)
[error] 72-72: Do not use sudo as it leads to unpredictable behavior. Use a tool like gosu to enforce root
(DL3004)