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

feat(coin): support nucleus as an alternative to iris HTLC #2079

Merged
merged 20 commits into from
Apr 18, 2024

Conversation

onur-ozkan
Copy link
Member

@onur-ozkan onur-ozkan commented Mar 4, 2024

This PR implements nucleus chain into mm2 providing an alternative HTLC backend to Iris.

@onur-ozkan onur-ozkan added enhancement New feature or request in progress Changes will be made from the author labels Mar 4, 2024
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
@onur-ozkan onur-ozkan marked this pull request as ready for review March 11, 2024 06:51
@onur-ozkan
Copy link
Member Author

This is almost ready to be reviewed/merged, with just a couple of tests needing to be updated. I will take care of this once the explorer is ready for Nucleus.

@onur-ozkan onur-ozkan added under review and removed in progress Changes will be made from the author labels Mar 11, 2024
Copy link
Collaborator

@shamardy shamardy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the PR! Please add doc comments to new files and also fix wasm tests!
I guess once tests are updated ref here #2079 (comment) this PR can be approved and merged.

@shamardy shamardy mentioned this pull request Apr 15, 2024
5 tasks
… into nucleus-support

Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
@onur-ozkan
Copy link
Member Author

This is ready to go.

cc @shamardy

Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
shamardy
shamardy previously approved these changes Apr 18, 2024
Copy link
Collaborator

@shamardy shamardy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great Work! Only one non-blocker!

@@ -1075,7 +1075,7 @@ fn test_withdraw_and_send_hd() {
const TX_HISTORY: bool = false;
const PASSPHRASE: &str = "tank abandon bind salon remove wisdom net size aspect direct source fossil";

let coins = json!([rick_conf(), tbtc_segwit_conf()]);
let coins = json!([rick_conf(), tbtc_segwit_conf(), eth_testnet_conf()]);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't have eth_testnet_conf() in dev, looks like a problem due to merging.

Signed-off-by: onur-ozkan <work@onurozkan.dev>
@onur-ozkan onur-ozkan merged commit d36f43d into dev Apr 18, 2024
22 of 25 checks passed
@onur-ozkan onur-ozkan deleted the nucleus-support branch April 18, 2024 06:31
dimxy added a commit to dimxy/komodo-defi-framework that referenced this pull request May 9, 2024
* dev:
  feat(app-dir): implement root application dir `.kdf` (KomodoPlatform#2102)
  fix tendermint fee calculation (KomodoPlatform#2106)
  update dockerfile (KomodoPlatform#2104)
  docs(README): remove outdated information from the README (KomodoPlatform#2097)
  fix(sia): fix sia compilation after hd wallet PR merge (KomodoPlatform#2103)
  feat(hd_wallet): utxo and evm hd wallet and trezor (KomodoPlatform#1962)
  feat(sia): initial Sia integration (KomodoPlatform#2086)
  fix(BCH): deserialize BCH header that uses KAWPOW version correctly (KomodoPlatform#2099)
  fix(eth_tests): remove ETH_DEV_NODE from tests (KomodoPlatform#2101)
  feat(coin): support nucleus as an alternative to iris HTLC (KomodoPlatform#2079)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request under review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants