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

feature: submit transaction #414

Merged
merged 2 commits into from
Feb 1, 2021
Merged

feature: submit transaction #414

merged 2 commits into from
Feb 1, 2021

Conversation

rhyslbw
Copy link
Contributor

@rhyslbw rhyslbw commented Jan 29, 2021

Context

Closes #10

Proposed Solution

Adds a mutation to submit a signed and serialised transaction to the network.
The mutation accepts the transaction data (CBOR hex) and returns the transaction
hash if accepted by the node. It supports submitting transactions against any of the current and upcoming eras.

@rhyslbw rhyslbw force-pushed the feature/submit-transaction branch 2 times, most recently from 03d936f to 8ecb326 Compare February 1, 2021 05:03
Adds a mutation to submit a signed and serialised transaction to the network.
The mutation accepts the transaction data (CBOR hex) and returns the transaction
hash if accepted by the node. It supports submitting transactions against any of the current and upcoming eras.
@rhyslbw rhyslbw force-pushed the feature/submit-transaction branch from 8ecb326 to 9e8e1cc Compare February 1, 2021 07:16
Copy link
Contributor

@ArturWieczorek ArturWieczorek left a comment

Choose a reason for hiding this comment

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

Tests on mainnet passed, submitTRansaction mutation returned hash of a transaction that I can now see in Deadalus, negative tests now return response from cli like UtxoFailure (ValueNotConservedUTxO or InvalidWitnessesUTXOW [VKey (VerKeyEd25519DSIGN (PublicKey "\183\138\&0\218\178\2.... ) which previously were not present.

@rhyslbw rhyslbw merged commit 37c6233 into master Feb 1, 2021
@rhyslbw rhyslbw deleted the feature/submit-transaction branch February 1, 2021 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add submit transaction mutation
2 participants