Skip to content

Update CLI guide content #951

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Dec 17, 2020
Merged

Update CLI guide content #951

merged 4 commits into from
Dec 17, 2020

Conversation

agraebe
Copy link
Contributor

@agraebe agraebe commented Dec 7, 2020

Replacing #927

@agraebe agraebe requested review from friedger and yknl December 7, 2020 22:38
@vercel
Copy link

vercel bot commented Dec 7, 2020 •
edited
Loading

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/blockstack/blockstack-docs/ggyzewnm4
✅ Preview: https://blockstack-docs-git-fix-cli-guide.blockstack.vercel.app

yknl
yknl reviewed Dec 8, 2020
View reviewed changes
src/pages/understand-stacks/command-line-interface.md Outdated
```bash
$ stx send_tokens ST2KMMVJAB00W5Z6XWTFPH6B13JE9RJ2DCSHYX0S7 1000 200 0 381314da39a45f43f45ffd33b5d8767d1a38db0da71fea50ed9508e048765cf301 -t

d32de0d66b4a07e0d7eeca320c37a10111c8c703315e79e17df76de6950c622c
Copy link
Contributor

Choose a reason for hiding this comment

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

We now respond with the txid and an explorer link

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good point, didn't look closely enough at this content

Copy link
Contributor Author

@agraebe agraebe Dec 9, 2020 •
edited
Loading

Choose a reason for hiding this comment

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

I realize the references arent' updated either: https://docs.blockstack.org/references/blockstack-cli#send_tokens

@yknl can you help fix as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ping

@agraebe
Copy link
Contributor Author

agraebe commented Dec 11, 2020

@yknl is this ready to be merged?

@yknl
Copy link
Contributor

yknl commented Dec 11, 2020

Yes, not sure why the vercel deployment failed though.

@agraebe
Copy link
Contributor Author

agraebe commented Dec 11, 2020

@aulneau could you take a look. it shows this as an error:

TypeError: Cannot set property 'customBlocks' of undefined

agraebe
agraebe commented Dec 15, 2020
View reviewed changes
@agraebe agraebe merged commit d9072c0 into master Dec 17, 2020
@agraebe agraebe deleted the fix/cli-guide branch December 17, 2020 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@friedger friedger Awaiting requested review from friedger

1 more reviewer

@yknl yknl yknl approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@agraebe @yknl