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

pcli: print height of detected transactions #2489

Merged
merged 2 commits into from
May 12, 2023
Merged

Conversation

hdevalence
Copy link
Member

This is useful for working with the DEX, where most of the queries are height-based.
Example output:

     Running `target/release/pcli -n 'http://testnet-preview.penumbra.zone:8080' tx lp order buy '20penumbra@0.9gn' --spread 50`
Scanning blocks from last sync height 522 to latest height 522
[0s] ██████████████████████████████████████████████████       0/0       0/s ETA: 0s
building transaction...
finished proving in 0.874 seconds [4 actions, 3 proofs, 2542 bytes]
broadcasting transaction and awaiting confirmation...
transaction confirmed and detected: 278a6b0a6e7781097551dff250094c0494c32569470bf4f6411c832865d3b68c @ 523

@plaidfinch
Copy link
Collaborator

Perhaps should we say @ block 523 to make it self evident what the number means?

@hdevalence
Copy link
Member Author

Any ideas why the smoke test is failing here? I can't tell from the error message.

@hdevalence hdevalence force-pushed the broadcast-return-height branch from 254e191 to cd5911d Compare May 12, 2023 00:18
@hdevalence hdevalence temporarily deployed to smoke-test May 12, 2023 00:18 — with GitHub Actions Inactive
@hdevalence hdevalence merged commit 4e7e590 into main May 12, 2023
@hdevalence hdevalence deleted the broadcast-return-height branch May 12, 2023 03:15
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.

2 participants