Skip to content
This repository has been archived by the owner on Aug 2, 2024. It is now read-only.

feat(rpc) implemented v0.5.1 missing fields + bumped starknet.rs to match #1313

Merged
merged 52 commits into from
Jan 10, 2024
Merged

Conversation

antiyro
Copy link
Contributor

@antiyro antiyro commented Dec 11, 2023

feat(rpc) implemented v0.5.1 missing fields + bumped starknet.rs to match

This pull request is to update the Madara code base to the latest version of RPC specs v0.5.1 as defined in here starknet api json. To achieve this, it was necessary to bump starknet.rs version to 64ebc36 commit in starknet-api and blockifier.

The fields l1_gas_price, message_hash, execution_resources has been defaulted since they're beeing implemented here

This pull request does not implement any changes but is needed to solve the v0.5.1 madara rpc upgrade

@antiyro
Copy link
Contributor Author

antiyro commented Dec 22, 2023

Some problems in this PR are still to be solved because of new fields/impl in starknet.rs:

  • FunctionInvocation(InvokeTransactionTrace): needs a refacto fo EventContent and MessageToL1to OrderedEvent and OrderedMessage
  • state_difffield missing in Transaction traces field (invoke, l1message, etc)

Implies changes in starknet-api.

More to be identified.

@antiyro
Copy link
Contributor Author

antiyro commented Dec 23, 2023

#1327
#1326
#1325

Should make this PR mergeable

@tdelabro tdelabro self-requested a review January 3, 2024 18:20
CHANGELOG.md Outdated Show resolved Hide resolved
crates/client/rpc-core/starknet_openRPC.json Outdated Show resolved Hide resolved
crates/primitives/transactions/src/execution.rs Outdated Show resolved Hide resolved
crates/pallets/starknet/runtime_api/Cargo.toml Outdated Show resolved Hide resolved
crates/primitives/fee/src/lib.rs Outdated Show resolved Hide resolved
crates/primitives/simulations/Cargo.toml Outdated Show resolved Hide resolved
crates/primitives/state/Cargo.toml Outdated Show resolved Hide resolved
crates/primitives/state/Cargo.toml Outdated Show resolved Hide resolved
crates/primitives/state/src/lib.rs Outdated Show resolved Hide resolved
crates/primitives/block/src/header.rs Outdated Show resolved Hide resolved
crates/primitives/block/src/lib.rs Outdated Show resolved Hide resolved
crates/primitives/commitments/Cargo.toml Show resolved Hide resolved
crates/primitives/simulations/Cargo.toml Outdated Show resolved Hide resolved
crates/primitives/state/Cargo.toml Outdated Show resolved Hide resolved
Copy link
Collaborator

@tdelabro tdelabro left a comment

Choose a reason for hiding this comment

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

lfg

@tdelabro tdelabro merged commit 5b31a4c into keep-starknet-strange:main Jan 10, 2024
12 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jan 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants