feat: update declaration of Braavos account in genesis

[lana-shanghai]

Feature Request

Describe the Feature Request

Currently in genesis we have these Braavos account class hashes

pub const BRAAVOS_ACCOUNT_CLASS_HASH_CAIRO_0: &str =
    "0x0244ca3d9fe8b47dd565a6f4270d979ba31a7d6ff2c3bf8776198161505e8b52";
pub const BRAAVOS_PROXY_CLASS_HASH_CAIRO_0: &str = "0x06a89ae7bd72c96202c040341c1ee422474b562e1d73c6848f08cae429c33262";

When we try to declare a new account using Braavos, we get errors:

{"jsonrpc":"2.0","error":{"code":40,"message":"Contract error"},"id":0}
{"jsonrpc":"2.0","error":{"code":20,"message":"Contract not found"},"id":0}

because the latest wallet expects a different class hash declared.

In Madara logs, the following class hash is not found 0x03ae692aaf1ded26a0b58cf42490f757563850acea887ed57b4894fee8279063

which is a Braavos account as seen on Starkscan

Compiled program link

Add this class hash and code to the genesis contracts deployed on Madara and verify the flow of setting up a new account on Braavos. This issue is also linked to transaction version which blocks both starknet foundry flow and Braavos, but the temporary removal of the is_query check allows to decouple the Braavos flow from starknet foundry.

[ftupas]

Hello, may I take this?

[lana-shanghai]

Please do!

[EvolveArt]

Hey @ftupas what's the status on this ?

[ftupas]

Hey there @EvolveArt, sorry I took a bit of time off to deal with personal things. I started to continue working on this, apologies for the delay

[github-actions[bot]]

There hasn't been any activity on this issue recently, and in order to prioritize active issues, it will be marked as stale. Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by leaving a 👍 Because this issue is marked as stale, it will be closed and locked in 7 days if no further activity occurs. Thank you for your contributions!

[apoorvsadana]

This was recently fixed but broke again with a new Braavos update. Assigning it to myself for now.

[github-actions[bot]]

There hasn't been any activity on this issue recently, and in order to prioritize active issues, it will be marked as stale. Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by leaving a 👍 Because this issue is marked as stale, it will be closed and locked in 7 days if no further activity occurs. Thank you for your contributions!

[tdelabro]

@apoorvsadana what is the status here?

[apoorvsadana]

Asked on the group, will update

[github-actions[bot]]

There hasn't been any activity on this issue recently, and in order to prioritize active issues, it will be marked as stale. Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by leaving a 👍 Because this issue is marked as stale, it will be closed and locked in 7 days if no further activity occurs. Thank you for your contributions!

[tdelabro]

@apoorvsadana can you document what to do?

[apoorvsadana]

Ya, so basically, we want to declare Braavos contracts on Madara and check if using the wallet extension are we able to use Braavos with Madara or not. If it works fine, we can add the new contracts to the genesis. I will ask @ocdbytes to pick it up.

[ocdbytes]

I will take this

[tdelabro]

@ocdbytes thanks a lot

[github-actions[bot]]

There hasn't been any activity on this issue recently, and in order to prioritize active issues, it will be marked as stale. Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by leaving a 👍 Because this issue is marked as stale, it will be closed and locked in 7 days if no further activity occurs. Thank you for your contributions!

[tdelabro]

repository archived in favor of https://github.com/madara-alliance/madara