-
Notifications
You must be signed in to change notification settings - Fork 16
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: update SDK based on new verification model #105
Conversation
CI failed |
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.
Hold as requested by @ashchan.
Is this going to |
That's the plan. |
BTW which is the upstream CKB pr on this change? |
Still working on it, ETA is this morning |
cba6a06
to
a16ab01
Compare
@@ -16,7 +16,7 @@ def initialize(host: CKB::RPC::DEFAULT_URL, mode: MODE::TESTNET) | |||
@rpc = CKB::RPC.new(host: host) | |||
if mode == MODE::TESTNET | |||
# Testnet system script code_hash | |||
expected_code_hash = "0x9e3b3557f11b2b3532ce352bfe8017e9fd11d154c4c7f9b7aaaa1e621b539a08" | |||
expected_code_hash = "0xa4a3d87cd69733562865ddfaf12dd44cf4cd396c4128ff1a5839c2fe48b2a1a1" |
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.
This seems to be changing again on CKB rc/v0.14.0
. Confirming that now.
https://github.com/nervosnetwork/ckb/pull/941/files#diff-346487b1ea6102562958846069e090deR9
Please hold on merging this for now till the PR in CKB repo is merged, tho we can get review process started early.