Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Update getSignatureStatus: support multiple signatures, include slot in each response item #9022

Merged

Conversation

CriesofCarrots
Copy link
Contributor

Problem

#8823 requires some blockstore changes. Rather than block on that rework, it would be nice to pick out the other new features of that PR to unblock downstream explorer work.

Summary of Changes

  • Update getSignatureStatus rpc to support querying for multiple signatures at once
  • Include slot transaction was processed in each response item

@garious
Copy link
Contributor

garious commented Mar 23, 2020

#8823 still needed?

@CriesofCarrots
Copy link
Contributor Author

#8823 still needed?

Yes. But it requires a change to the Blockstore TransactionStatus column, which I will PR separately.

@CriesofCarrots CriesofCarrots merged commit 1b8f9e7 into solana-labs:master Mar 23, 2020
mvines pushed a commit to mvines/solana that referenced this pull request Mar 23, 2020
…in each response item (solana-labs#9022)

* Rename enable-rpc-get-confirmed-block

* Rename RpcTransactionStatus -> RpcTransactionStatusMeta

* Return simplified RpcTransactionStatus; Add support for multiple transactions

* Update docs

* typo
@mvines mvines mentioned this pull request Mar 23, 2020
mvines pushed a commit to mvines/solana that referenced this pull request Mar 23, 2020
…in each response item (solana-labs#9022)

* Rename enable-rpc-get-confirmed-block

* Rename RpcTransactionStatus -> RpcTransactionStatusMeta

* Return simplified RpcTransactionStatus; Add support for multiple transactions

* Update docs

* typo
@CriesofCarrots CriesofCarrots deleted the update-get-signature-status branch April 16, 2020 17:44
danpaul000 pushed a commit to danpaul000/solana that referenced this pull request Jul 8, 2020
…in each response item (solana-labs#9022)

* Rename enable-rpc-get-confirmed-block

* Rename RpcTransactionStatus -> RpcTransactionStatusMeta

* Return simplified RpcTransactionStatus; Add support for multiple transactions

* Update docs

* typo
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.

3 participants