-
Notifications
You must be signed in to change notification settings - Fork 67
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
Doc comments 1 #837
Doc comments 1 #837
Conversation
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.
Love the links to the EIPs. 👍
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.
For some of this documentation it would be even nicer (the PR is good too, I'm just thinking out loud) if we had a place on https://cow.fi/ to link to because technical users of the exchange or the api will have the same question and it would be better to have the information consolidated in one place.
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.
Looks good, thanks for the additional documentation!
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.
Great
@@ -6,6 +6,9 @@ use std::{ | |||
str::FromStr, | |||
}; | |||
|
|||
/// This allows arbitrary user data to be associated with an order. This type holds the | |||
/// hash of the data, while the data itself is uploaded to IPFS. The hash is signed along with 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.
Note that this isn't enforced.
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.
AWESOME!
Co-authored-by: Federico Giacon <[email protected]>
Co-authored-by: Federico Giacon <[email protected]>
e3cbce4
to
854ce95
Compare
Added some doc comments explaining things I was curious about.
Please let me know if I got anything wrong 😄