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

Edit Transactions description #558

Merged
merged 3 commits into from
Feb 25, 2022
Merged

Edit Transactions description #558

merged 3 commits into from
Feb 25, 2022

Conversation

Clay-Mysten
Copy link
Contributor

Standardize on some spellings, make small improvements, add some TODOs

Standardize on some spellings, make small improvements, add some TODOs
Adding BCS link
Copy link
Contributor

@huitseeker huitseeker left a comment

Choose a reason for hiding this comment

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

👍 thanks!

doc/transactions.md Outdated Show resolved Hide resolved
@@ -2,7 +2,9 @@

All updates to the Sui ledger happen via a transaction. This section describes the transaction types supported by Sui and explains how their execution changes the ledger.

## Transaction Metadata
TODO: Define *Gas* and settle upon spelling and capitalization right here.
Copy link
Contributor

Choose a reason for hiding this comment

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

Given #563 I'm getting a sense this is calling for a wandering @awelc :)

Co-authored-by: François Garillot <4142+huitseeker@users.noreply.github.com>
@Clay-Mysten Clay-Mysten merged commit 5a1e2e7 into main Feb 25, 2022
@Clay-Mysten Clay-Mysten deleted the Clay-Mysten-patch-4 branch February 25, 2022 17:26
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