-
Notifications
You must be signed in to change notification settings - Fork 4
feat: support testnets #472
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
base: main
Are you sure you want to change the base?
Conversation
|
Checked with starknet blocks post v0.14.0 upgrade on testnet and mainnet (mainnet seems to be halted right now. so tried with blocks till the halted height 1962873). there is no need for refactoring at our end for now. but the block time has been reduced to 6 secs from 30 secs. we need to update the default |
|
we need to use 0.9 RPC endpoints, as that became standard in starknet 0.14.0. but I couldn't find a working RPC endpoint post-0.14.0 upgrade. |
|
Looks like, |
|
Hacking around dependencies and proxying RPC endpoints, I managed to get a concrete error from Starknet about |
|
Looks like there is a bug in our code. ibc-starknet/cairo-contracts/packages/core/src/connection/components/handler.cairo Lines 251 to 257 in f85d4e1
It should use the connection ID of the counterparty at chain-B, which is the connection ID at chain-A. let expected_conn_end_on_b = ConnectionEndTrait::try_open(
conn_end_on_a.counterparty.client_id.clone(),
conn_end_on_a.client_id.clone(),
- msg.conn_id_on_b.clone(),
+ msg.conn_id_on_a.clone(),
conn_end_on_a.counterparty.prefix.clone(),
conn_end_on_a.delay_period,
); With this change, the bootstrap works without the ps. to patch this, I upgraded the contract with the newer class hash. thus, I managed to test contract upgradability -- which works alright. opened #473 |
|
I am getting an occasional error while batching: |
closes #440
closes #468
This is a feature branch to test our work on testnets manually.
This is required as
madara(our e2e test devnet) is outdated than the starknet testnet.