Conversation
AurelienGasser
requested review from
camillemarini,
inelgnu and
thibaultrobert
as code owners
|
|
||
| Note for internal use only: See the [technical specifications](https://github.com/SubstraFoundation/substra-spec/blob/master/technical_spec_substra.md#smartcontract). | ||
|
|
There was a problem hiding this comment.
Maybe add a link to https://github.com/SubstraFoundation/substra-doc?
There was a problem hiding this comment.
substra-doc is now a legacy repo ;) The user doc is located in the substra repo.
There was a problem hiding this comment.
I forgot about that. So let's send him to substra then.
There was a problem hiding this comment.
@thibaultrobert What should the text for the link say?
There was a problem hiding this comment.
Well my first guess would be https://github.com/SubstraFoundation/substra but maybe @natct10 has a better suggestion as to where we can find Substra documentation.
There was a problem hiding this comment.
@thibaultrobert Sorry, I meant which text should we add a link to? like [what do we put here???](some_link_to_susbtra)
There was a problem hiding this comment.
Ok Mondays and I don't agree with each others.... [Substra main repository] ?
But at this point I would rather agree with the current state of the PR and remove that part completely. Maybe add a link to Substra but not as a Note for internal use only. So let me approve that for you and sorry for the noise
There was a problem hiding this comment.
Hi, a small piece of documentation related to smart contracts is on its way (but not ready yet!), what do you think about temporarily removing this internal link & redirecting towards the general doc and update this part later?
|
@AurelienGasser Any update on this one ? |
5 participants
What do you think of removing this link that 404s for people who don't have access to the private repo substra-specs?