Skip to content
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(rpc): Added starknet_getTransactionStatus and removed starknet_pendingTransactions #1299

Merged
merged 23 commits into from
Dec 6, 2023
Merged

feat(rpc): Added starknet_getTransactionStatus and removed starknet_pendingTransactions #1299

merged 23 commits into from
Dec 6, 2023

Conversation

antiyro
Copy link
Contributor

@antiyro antiyro commented Dec 6, 2023

feat(rpc): Added starknet_getTransactionStatus and removed starknet_pendingTransactions

This PR solves the implementation of a new call: starknet_getTransactionStatus and the removal of starknet_pendingTransactions according to the v0.5.1 RPC specs described in #1291

starknet_getTransactionStatus

Gets the Transaction Status, Including Mempool Status and Execution Details

This method retrieves the status of a specified transaction. It provides information on
whether the transaction is still in the mempool, has been executed, or dropped from the
mempool. The status includes both finality status and execution status of the
transaction.

Arguments

  • transaction_hash - The hash of the transaction for which the status is requested.

Returns

  • transaction_status - An object containing the transaction status details:
  • finality_status: The finality status of the transaction, indicating whether it is
    confirmed, pending, or rejected.
  • execution_status: The execution status of the transaction, providing details on the
    execution outcome if the transaction has been processed.

Expected behavior

image

Notes

crates/client/rpc/src/lib.rs Outdated Show resolved Hide resolved
crates/client/rpc/src/lib.rs Outdated Show resolved Hide resolved
crates/client/rpc/src/lib.rs Outdated Show resolved Hide resolved
crates/client/rpc/src/lib.rs Outdated Show resolved Hide resolved
crates/client/rpc/src/lib.rs Outdated Show resolved Hide resolved
crates/client/rpc/src/lib.rs Outdated Show resolved Hide resolved
crates/client/rpc/src/lib.rs Outdated Show resolved Hide resolved
crates/client/rpc/src/lib.rs Show resolved Hide resolved
@tdelabro tdelabro merged commit 17f8a0f into keep-starknet-strange:main Dec 6, 2023
12 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Dec 8, 2023
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.

None yet

2 participants