Merged
Conversation
* added pos mode v1 * add pos service * format * fix yellow underline * Payment mode says 'Pay' * format * add watch * add changes back in * new charge fix, printouts, wait button added * copy button for debug * update rust crates to new chain version (planck) * minor fix for dev accounts * formatting only - no changes * comment out debug button * Addressing some issues
* feat: update cahin history graphql query * feat: update extrinsicHash accessor from graphql data
Add known value test for wormhole
dewabisma
approved these changes
Apr 14, 2026
Collaborator
dewabisma
left a comment
dewabisma
left a comment
There was a problem hiding this comment.
I have faith it works as expected 🤞
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
new testnet is LIVE
Known Value tests fix
Root cause: The SDK's
to_account_idwas usingPoseidonHasher::hash(which internally callshash_for_circuit- a ZK-circuit-optimized hash with compact encoding and padding). The chain useshash_bytes(injective encoding, 4 bytes/felt, no padding). These are explicitly different functions - theqp-poseidon-coretest suite even asserts they produce different output.Changes made:
rust/Cargo.toml- Replacedqp-poseidonwithqp-poseidon-core(direct dependency on the core crate that hashash_bytes)rust/src/api/crypto.rs- Changedto_account_idto usehash_bytes(matching the chain'sAccountId32::new(hash_bytes(pubkey))) instead ofPoseidonHasher::hashtest/generate_keys_test.dart- Uncommented the master key assertion, updated the "known values2" expected address to match the corrected outputNew migration dialog
Tested error cases too