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

cleaned up commands,outputs #545

Merged
merged 1 commit into from
Mar 9, 2022

Conversation

BharatAtEMURGOIO
Copy link
Contributor

Updating documentation

Description of the change

  1. Cleaned up the xxd command to remove -b option (incompatible with -ps option) and unnecessary command option -c 80 as the output token name is less than 40 characters
  2. Added env. variable for '--testnet-magic 1097911063' to help reduce clutter in the commands.
  3. Corrected display of cardano-cli query utxo command as cardano-cli no longer shows the 'Testtoken'/'SecondTestToken' text, instead it displays the base16 encoded token names.
  4. added some minor emphasis on certain text to highlight important points.

1. cleaned up the xxd command to remove -b option (incompatible with -ps) and unnecessary command -c 80 as the output token name is less than 80 characters
2. also added env. variable for '--testnet-magic 1097911063' to help reduce clutter in the commands.
3. Corrected display of cardano-cli query utxo command as cardano-cli no longer shows the testtoken text, instead it displays the base16 encoded token names.
4. added some minor emphasis on certain text to highlight important points.
Copy link
Collaborator

@gufmar gufmar left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@katomm katomm left a comment

Choose a reason for hiding this comment

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

👍

@katomm katomm merged commit d5c6484 into cardano-foundation:staging Mar 9, 2022
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.

4 participants