Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

[docs]command ref docs fix - fix broken link, replace Set, fix callout #10362

Merged
merged 1 commit into from
May 17, 2021
Merged

[docs]command ref docs fix - fix broken link, replace Set, fix callout #10362

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion docs/02_cleos/03_command-reference/set/set-account-permission.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,8 @@ The following information shows the different positionals and options you can us

## Requirements
* Install the currently supported version of `cleos`.
[[info | Note]] | `Cleos` is bundled with the EOSIO software. [Installing EOSIO](../../00_install/index.md) will also install the `cleos` and `keosd` comand line tools.
[[info | Note]]
| `Cleos` is bundled with the EOSIO software. [Installing EOSIO](../../../00_install/index.md) will also install the `cleos` and `keosd` comand line tools.
Copy link
Contributor

Choose a reason for hiding this comment

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

cleos is a program name. @bobgt Is it okay for system names (which we wrap in backticks as a convention) to use sentence case (as in Cleos) even when starting a sentence?

* You have access to an EOSIO blockchain.
* You have an EOSIO account and access to the account's private key.

Expand Down
5 changes: 3 additions & 2 deletions docs/02_cleos/03_command-reference/set/set-action-permission.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,8 @@ The following information shows the different positionals and options you can us

## Requirements
* Install the currently supported version of `cleos`.
[[info | Note]] | `Cleos` is bundled with the EOSIO software. [Installing EOSIO](../../00_install/index.md) will also install the `cleos` and `keosd` comand line tools.
[[info | Note]]
| `Cleos` is bundled with the EOSIO software. [Installing EOSIO](../../../00_install/index.md) will also install the `cleos` and `keosd` comand line tools.
* You have access to an EOSIO blockchain.
* You have an EOSIO account and access to the account's private key.

Expand All @@ -64,7 +65,7 @@ executed transaction: 4eb4cf3aea232d46e0e949bc273c3f0575be5bdba7b61851ab51d927cf
# eosio <= eosio::linkauth {"account":"bob","code":"scontract1","type":"hi","requirement":"customp1"}
```

2. permissions on a _bye_ action deployed to the _scontract1_ account so that the _bob_ account's `active` permission and _customp2_ permission are authorized:
2. Set permissions on a _bye_ action deployed to the _scontract1_ account so that the _bob_ account's `active` permission and _customp2_ permission are authorized:
Copy link
Contributor

Choose a reason for hiding this comment

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

I still don't understand why active and customp2 have to use different formatting styles. Both are permissions and they are both instantiated from the same class.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The idea is to differentiate between defaut and custom permissions to emphsise this for this how to


```shell
cleos set action permission bob scontract1 bye customp2 -p bob@active
Expand Down