Remove mistakenly merged files. Add code references, make title consi…#4
Remove mistakenly merged files. Add code references, make title consi…#4
Conversation
…stent across themis, remove reference to a meeting notes doc Signed-off-by: Lovesh Harchandani <lovesh.bond@gmail.com>
| @@ -1,4 +1,4 @@ | |||
| # CRED_OFFER | |||
| # Credential Offer | |||
|
|
|||
There was a problem hiding this comment.
We need both long names (e.g., Credential Offer) as well as msg types (e.g., CRED_OFFER) right?
There was a problem hiding this comment.
Currently there is nothing called a CRED_OFFER in the open source codebase atleast
| ``` | ||
|
|
||
|
|
||
| ## Code reference (some of below might be different from above mentioned structure, the above structure should be considered the correct representation) |
There was a problem hiding this comment.
Why might it be different? It seems that the code would be more correct.
There was a problem hiding this comment.
The code would (and in some cases "should") be refactored.
There was a problem hiding this comment.
OK, so what should we do when we have documentation that is ahead of the code structure? I think the documentation should mention both the current version, and the future version, right?
|
|
||
| ## Code references to ledger objects | ||
| ## Code references to ledger objects (some of below might be different from above mentioned structure, the above structure should be considered the correct representation) | ||
| 1. [CLAIM_DEF](https://github.com/hyperledger/indy-sdk/blob/778a38d92234080bb77c6dd469a8ff298d9b7154/libindy/src/services/ledger/types.rs#L234) |
There was a problem hiding this comment.
Shouldn't this be CRED_DEF?
There was a problem hiding this comment.
OK, so you'll make that change?
| ## Code references to ledger objects | ||
| ## Code references to ledger objects (some of below might be different from above mentioned structure, the above structure should be considered the correct representation) | ||
| 1. [CLAIM_DEF](https://github.com/hyperledger/indy-sdk/blob/778a38d92234080bb77c6dd469a8ff298d9b7154/libindy/src/services/ledger/types.rs#L234) | ||
| 2. [Primary public key](https://github.com/hyperledger/indy-crypto/blob/fc078a014a6b72ede838b79696258d3ee56f87d4/libindy-crypto/src/cl/mod.rs#L161) |
There was a problem hiding this comment.
This doesn't seem like protocol level documentation. Maybe this should be in the indy-crypto repo itself?
There was a problem hiding this comment.
Yes, why a credential definition has primary and revocation sections go in this protocol repo and the details of the structure of each of the objects go in the respective source code repo (indy-crypto, indy-sdk, indy-node)
| ## Code references to ledger objects (some of below might be different from above mentioned structure, the above structure should be considered the correct representation) | ||
| 1. [CLAIM_DEF](https://github.com/hyperledger/indy-sdk/blob/778a38d92234080bb77c6dd469a8ff298d9b7154/libindy/src/services/ledger/types.rs#L234) | ||
| 2. [Primary public key](https://github.com/hyperledger/indy-crypto/blob/fc078a014a6b72ede838b79696258d3ee56f87d4/libindy-crypto/src/cl/mod.rs#L161) | ||
| 3. [Revocation public key](https://github.com/hyperledger/indy-crypto/blob/fc078a014a6b72ede838b79696258d3ee56f87d4/libindy-crypto/src/cl/mod.rs#L202) |
There was a problem hiding this comment.
These links will work forever, which is great.
Problem is, if that object evolves, then the documentation won't show that.
The reason for having the link is we can jump to the current code representation of that object.
Maybe we should move some of this lower level stuff to the source code repo and reference the documentation from this repo?
There was a problem hiding this comment.
Ok, if we move the sections describing the structure of Cred def, etc to respective source repos, then these can be converted to links to master (active) branch and can be updated with the code
2 participants
…stent across themis, remove reference to a meeting notes doc
Signed-off-by: Lovesh Harchandani lovesh.bond@gmail.com