Skip to content
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

Update netid, coins and bump app version #88

Merged
merged 5 commits into from
Dec 18, 2023
Merged

Conversation

CharlVS
Copy link
Member

@CharlVS CharlVS commented Dec 17, 2023

  • Update Net ID

  • Major (semver) release version because net ID update is a breaking change since users on old net ID cannot see the order book of new net ID users. This is part of a p2p network stack upgrade for Komodo DeFi framework, which opens up future possibilities for more efficient streaming-based event features.

  • Update coin config and icon files. Even though coin config files are updated at runtime, it’s a good idea to keep the bundled version up to date since it’s used on the first launch and is used as a fallback if the update fails. Also, some coin authors for new coins have been waiting for us to push an update because their icon is missing from the app because icons are not updated at runtime.
    NB: In the near future, coin config and icon files will be dropped from the repo and instead fetched at CI build time in PR Optimise coins asset files #87

  • Upgrade min Android SDK version to fix failed builds. New DeFi version requires newer Android NDK version: https://github.com/KomodoPlatform/komodo-defi-framework/blob/092d562f476ece1bef3cf3e8e50694e23688dade/scripts/ci/android-ndk.sh#L5-L6

Even though coin config files are updated at runtime, it’s a good idea to keep the bundled version up to date since it’s used on the first launch and is used as a fallback if update fails.

NB: In the near future, these files will be dropped from the repo and instead rather fetched at CI build time in PR #87 (#87).
Major release version because net ID update is a breaking change since users on old net ID cannot see order book of new net ID users. This is part of a p2p network stack upgrade for Komodo DeFi framework which opens up future possibilities for more efficient streaming-based event features.
@CharlVS CharlVS added the QA Ready for QA testing label Dec 17, 2023
@CharlVS CharlVS self-assigned this Dec 17, 2023
@CharlVS CharlVS changed the title Update netid, coins and bump app version Update netid, coins and bump app version (0.9.0) Dec 17, 2023
@CharlVS CharlVS changed the title Update netid, coins and bump app version (0.9.0) Update netid, coins and bump app version Dec 17, 2023
The new Komodo DeFi Framework p2p layer changes require Android NDK >=23.
@CharlVS CharlVS added the 0.9.0 v0.9.0 release: Net ID migration and order book + file export bug fixes. label Dec 17, 2023
@ca333 ca333 merged commit a9fa266 into dev Dec 18, 2023
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.9.0 v0.9.0 release: Net ID migration and order book + file export bug fixes. QA Ready for QA testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants