-
Notifications
You must be signed in to change notification settings - Fork 26
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
Comment PR #502
base: empty
Are you sure you want to change the base?
Comment PR #502
Conversation
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Co-authored-by: William Richter <88400283+WRRicht3r@users.noreply.github.com> Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
* [documentation]: Review Triggers chapter Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com> * Fix formatting Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com> * Apply suggestions from code review Co-authored-by: William Richter <88400283+WRRicht3r@users.noreply.github.com> Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com> * Fix formatting Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com> * Add a link to queries Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com> Co-authored-by: William Richter <88400283+WRRicht3r@users.noreply.github.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Co-authored-by: William Richter <88400283+WRRicht3r@users.noreply.github.com> Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Co-authored-by: William Richter <88400283+WRRicht3r@users.noreply.github.com> Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Co-authored-by: William Richter <88400283+WRRicht3r@users.noreply.github.com> Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Co-authored-by: Aleksandr Petrosyan <a-p-petrosyan@yandex.ru> Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Co-authored-by: William Richter <88400283+WRRicht3r@users.noreply.github.com> Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Co-authored-by: Aleksandr Petrosyan <a-p-petrosyan@yandex.ru> Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: 0x009922 <a.marcius26@gmail.com>
* ADD: Table of tokens Signed-off-by: Stukalov-A-M <stukalov07@gmail.com> * [EDIT] #460,#461,#462 : Permissions example Signed-off-by: Stukalov-A-M <stukalov07@gmail.com> * [EDIT] #460,#461,#462 : Permissions example; Metadata links to permissions updated. Signed-off-by: Stukalov-A-M <stukalov07@gmail.com> * [EDIT] #460,#461,#462 : Permissions example; Metadata links to permissions updated. Signed-off-by: Stukalov-A-M <stukalov07@gmail.com> * [EDIT] #460,#461,#462 : Permissions example; Metadata links to permissions updated. Signed-off-by: Stukalov-A-M <stukalov07@gmail.com> * [EDIT] #460,#461,#462 : Permissions example; Metadata links to permissions updated. Signed-off-by: Stukalov-A-M <stukalov07@gmail.com> * Update src/guide/blockchain/metadata.md Co-authored-by: Ekaterina Mekhnetsova <mekkatya@gmail.com> Signed-off-by: Stukalov-A-M <68690788+Stukalov-A-M@users.noreply.github.com> * Update src/reference/permissions.md Co-authored-by: Ekaterina Mekhnetsova <mekkatya@gmail.com> Signed-off-by: Stukalov-A-M <68690788+Stukalov-A-M@users.noreply.github.com> * Update src/reference/permissions.md Co-authored-by: Ekaterina Mekhnetsova <mekkatya@gmail.com> Signed-off-by: Stukalov-A-M <68690788+Stukalov-A-M@users.noreply.github.com> * Update src/reference/permissions.md Co-authored-by: Ekaterina Mekhnetsova <mekkatya@gmail.com> Signed-off-by: Stukalov-A-M <68690788+Stukalov-A-M@users.noreply.github.com> --------- Signed-off-by: Stukalov-A-M <stukalov07@gmail.com> Signed-off-by: Stukalov-A-M <68690788+Stukalov-A-M@users.noreply.github.com> Co-authored-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
* Updated and polished Torii Endpoints; fixed and/or removed dead links in related topics Signed-off-by: yamkovoy <b.yamkovoy@gmail.com> * Apply suggestions from code review Co-authored-by: 0x009922 <43530070+0x009922@users.noreply.github.com> Signed-off-by: yamkovoy <100563712+yamkovoy@users.noreply.github.com> * Updated after review Signed-off-by: yamkovoy <b.yamkovoy@gmail.com> * Apply suggestions from code review Co-authored-by: 0x009922 <43530070+0x009922@users.noreply.github.com> Signed-off-by: yamkovoy <100563712+yamkovoy@users.noreply.github.com> * Torii: Updated after review, minor fixes Signed-off-by: yamkovoy <b.yamkovoy@gmail.com> --------- Signed-off-by: yamkovoy <b.yamkovoy@gmail.com> Signed-off-by: yamkovoy <100563712+yamkovoy@users.noreply.github.com> Co-authored-by: 0x009922 <43530070+0x009922@users.noreply.github.com>
* [chore]: update deps Signed-off-by: Dmitry Balashov <43530070+0x009922@users.noreply.github.com> * [chore]: `type: module`, fixing warning from Vitest Signed-off-by: Dmitry Balashov <43530070+0x009922@users.noreply.github.com> * [revert]: project is not ready for ESM Signed-off-by: Dmitry Balashov <43530070+0x009922@users.noreply.github.com> --------- Signed-off-by: Dmitry Balashov <43530070+0x009922@users.noreply.github.com>
Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
…483) * Updated Rust guide to include information about transferring assets Signed-off-by: enxtur <enxtur.mon@gmail.com> * Apply suggestions from code review Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com> * Update instructions.md Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com> * Update instructions.md Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com> --------- Signed-off-by: enxtur <enxtur.mon@gmail.com> Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com> Co-authored-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
Signed-off-by: Nikita Strygin <dcnick3@users.noreply.github.com>
…g paths Signed-off-by: Nikita Strygin <dcnick3@users.noreply.github.com>
…4544 Signed-off-by: Nikita Strygin <dcnick3@users.noreply.github.com>
Signed-off-by: Nikita Strygin <dcnick3@users.noreply.github.com>
Signed-off-by: Nurzhan Sakén <nurzhan.sakenov@gmail.com>
* Replace `docker-compose` (V1) with `docker compose` (V2) * Fix spelling in bare metal tutorial * Update src/guide/advanced/running-iroha-on-bare-metal.md --------- Signed-off-by: Nurzhan Saken <nurzhan.sakenov@gmail.com> Signed-off-by: yamkovoy <100563712+yamkovoy@users.noreply.github.com> Co-authored-by: yamkovoy <100563712+yamkovoy@users.noreply.github.com>
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
src/guide/blockchain/data-model.md
Outdated
``` | ||
|
||
+-----------------------------------------------+ | ||
| | | ||
| +-----------------+ | | ||
| |Domain | | | ||
| +--------------+ | | | ||
| ||Asset | | | | ||
+--+--+ ||Definition(s)| | | | ||
|World| +--------------+ | | | ||
+--+--+ | | | | ||
| +------------+ | | | ||
| ||Account(s)|| | has +-----------+ | | ||
| |------------------------->Signatories| | | ||
| +-----------------+ +-----------+ | | ||
| | | | ||
| | has +--------+ | | ||
| +------->Asset(s)| | | ||
| +--------+ | | ||
+-----------------------------------------------+ | ||
|
||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is changing to a flat model implementation-wise: hyperledger-iroha/iroha#3921, but this diagram is conceptually still valid. Except I believe an account can only have one signatory now: hyperledger-iroha/iroha#4411
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I think it is a good point to consider keeping conceptual diagrams:
In the future, internal representation of objects may diverge from their appearance at user interface.
For example, the primary key for account storage might be signatory
, as opposed to the (UI) account ID signatory@domain
, which is btw an intent of this suggestion.
However, it is not the implementation details that should be included in the user guide, but the conceptual diagram at UI
src/guide/blockchain/assets.md
Outdated
The assets can be **fungible** (every £1 is exactly the same as every other | ||
£1) or **non-fungible** (a £1 bill signed by the Queen of Hearts is not the | ||
same as a £1 bill signed by the King of Spades). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think asset definitions work like that. We don't have NFTs, only numeric assets (with elastic or fixed supply) and store assets (arbitrary JSON data even within a single definition)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My understanding is that Store
/Numeric
corresponds to NFT/FT, respectively
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@s8sato I was under the same impression, but then realized that true NFTs should disallow instantiating more than one of an asset definition (monalisa#louvre#alice@wland
and monalisa#louvre#bob@wland
cannot co-exist since there is only one Mona Lisa in Louvre).
BTW, I think that Store assets, as designed right now, don't make much sense because two instances of the same definition can have totally different keys.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed, we might move away from the idea of representing Store
by a combination of a shared definition and its own value.
Maybe ideally the identifier should be smt like a hash of the value.
We may move this discussion somewhere else, maybe hyperledger-iroha/iroha#4087 or hyperledger-iroha/iroha#4782 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apart from that, as for this doc, what do you think about not mentioning to FT/NFT but just mentioning it is planned to be implemented
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can omit NFT/FT and focus on the distinction between Numeric (with elastic or fixed global supply) and Store assets for now
src/guide/blockchain/assets.md
Outdated
same as a £1 bill signed by the King of Spades). | ||
|
||
The assets can also be **mintable** (you can make more of them) and | ||
**non-mintable** (you can only specify their initial quantity in the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Assets with a fixed supply can be minted once, but it doesn't have to be in the genesis. We should also rethink "non-mintable" as it really means "mintable but only once"
src/guide/blockchain/assets.md
Outdated
Additionally, the assets have different underlying value types. | ||
Specifically, we have `AssetValueType.Quantity`, which is effectively an | ||
unsigned 32-bit integer, a `BigQuantity`, which is an unsigned 128-bit | ||
integer, and `Fixed`, which is a positive (though signed) 64-bit | ||
fixed-precision number with nine significant digits after the decimal | ||
point. All three types can be registered as either **mintable** or | ||
**non-mintable**. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We only have Numeric
now, which is backed by the rust_decimal
crate.
src/guide/blockchain/assets.md
Outdated
There is also the `Store` asset type, which is used for storing key-values | ||
in object's metadata. We talk in detail about `Store` asset in the chapter | ||
related to [metadata](metadata.md). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A bit confusing. Store assets contain Metadata
(a Metadata
is just a mapping between keys and JSON strings, IMO we should rename this to smth like JsonMap
), and most objects also contain separate Metadata
(same key-JSON map type) but unrelated to assets.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, "metadata" sounds secondary, but it should be primary in Store
assets
src/guide/blockchain/expressions.md
Outdated
@@ -0,0 +1,17 @@ | |||
# Expressions, Conditionals, Logic |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Outdated section. This was replaced by Custom Instructions, which should be documented with the Executor probably.
src/guide/blockchain/instructions.md
Outdated
| [Grant/Revoke](#grant-revoke) | Give or remove certain permissions from accounts. | | ||
| [Transfer](#transfer) | Transfer assets between accounts. | | ||
| [ExecuteTrigger](#executetrigger) | Execute triggers. | | ||
| [If, Pair, Sequence](#composite-instructions) | Use to create composite instructions. | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed
src/guide/blockchain/metadata.md
Outdated
|
||
- domains | ||
- accounts | ||
- assets |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only store assets
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, but it's not exactly what attached to the main object
src/guide/blockchain/metadata.md
Outdated
The metadata can be of very different types, such as: | ||
|
||
- structures with named or unnamed fields | ||
- enums | ||
- integers | ||
- numbers with fixed decimal precision | ||
- strings | ||
- Boolean values | ||
- arrays | ||
- associative arrays | ||
- vectors | ||
- request results |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Metadata maps keys to JSON strings: hyperledger-iroha/iroha#4353
src/guide/blockchain/world.md
Outdated
- Iroha [configuration parameters](./../configure/client-configuration.md) | ||
- the list of | ||
[trusted peers](/guide/configure/peer-configuration#trusted-peers) | ||
- registered domains |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These have been unnested from domains, see hyperledger-iroha/iroha#3921:
- accounts
- asset definitions
- assets
src/guide/blockchain/wasm.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be renamed according to hyperledger-iroha/iroha#4810, see the change in the README for smart contracts
* Getting started update and overhaul Signed-off-by: a_zorina <64249490+a-zorina@users.noreply.github.com> Signed-off-by: a-zorina <64249490+a-zorina@users.noreply.github.com> * fix broken links Signed-off-by: a_zorina <64249490+a-zorina@users.noreply.github.com> Signed-off-by: a-zorina <64249490+a-zorina@users.noreply.github.com> * fix section links Signed-off-by: a_zorina <64249490+a-zorina@users.noreply.github.com> Signed-off-by: a-zorina <64249490+a-zorina@users.noreply.github.com> * Language and structure improvements Signed-off-by: a-zorina <64249490+a-zorina@users.noreply.github.com> * External links and crate names fixes Signed-off-by: a-zorina <64249490+a-zorina@users.noreply.github.com> * Switch to using cargo install for iroha binaries in getting started Signed-off-by: a-zorina <64249490+a-zorina@users.noreply.github.com> * Minor post-review fixes Signed-off-by: a-zorina <64249490+a-zorina@users.noreply.github.com> * Fixes and suggestions after review Signed-off-by: yamkovoy <b.yamkovoy@gmail.com> * Changes after discussion Signed-off-by: yamkovoy <b.yamkovoy@gmail.com> * Minor tweaks in instructions Signed-off-by: yamkovoy <b.yamkovoy@gmail.com> * Additional post-review updates Signed-off-by: a-zorina <64249490+a-zorina@users.noreply.github.com> * link fixes Signed-off-by: a-zorina <64249490+a-zorina@users.noreply.github.com> * final post-review changes Signed-off-by: a-zorina <64249490+a-zorina@users.noreply.github.com> --------- Signed-off-by: a_zorina <64249490+a-zorina@users.noreply.github.com> Signed-off-by: a-zorina <64249490+a-zorina@users.noreply.github.com> Signed-off-by: yamkovoy <b.yamkovoy@gmail.com> Co-authored-by: yamkovoy <b.yamkovoy@gmail.com>
* Update new structure with recent changes Signed-off-by: a-zorina <64249490+a-zorina@users.noreply.github.com> * tweaks to structure after discussions Signed-off-by: a-zorina <64249490+a-zorina@users.noreply.github.com> * tweaks to Iroha Explained section Signed-off-by: a-zorina <64249490+a-zorina@users.noreply.github.com> * format fix Signed-off-by: a-zorina <64249490+a-zorina@users.noreply.github.com> --------- Signed-off-by: a-zorina <64249490+a-zorina@users.noreply.github.com>
* link fixes due to migration Signed-off-by: a-zorina <64249490+a-zorina@users.noreply.github.com> * Update src/guide/tutorials/rust.md Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com> --------- Signed-off-by: a-zorina <64249490+a-zorina@users.noreply.github.com> Signed-off-by: Ekaterina Mekhnetsova <mekkatya@gmail.com> Co-authored-by: Ekaterina Mekhnetsova <mekkatya@gmail.com>
misc: update dependencies Signed-off-by: qua <43530070+0x009922@users.noreply.github.com>
PR to hopefully use for comments. Do not merge this.