-
Notifications
You must be signed in to change notification settings - Fork 277
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
[feature] #1606: Add ipfs link to domain logo in Domain structure #1886
[feature] #1606: Add ipfs link to domain logo in Domain structure #1886
Conversation
Codecov Report
@@ Coverage Diff @@
## iroha2-dev #1886 +/- ##
===========================================
Coverage 78.66% 78.67%
===========================================
Files 156 156
Lines 22140 22227 +87
===========================================
+ Hits 17417 17486 +69
- Misses 4723 4741 +18
Continue to review full report at Codecov.
|
data_model/src/lib.rs
Outdated
let _domain = Domain { | ||
logo: Some(logo), | ||
..Domain::test("sora") | ||
}; |
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.
there are no assertions in the test? does it test anything?
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 I would remove it
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.
Well, it just an example of usage with logo. But I'm agree, looks strange. I'm gonna remove it
…ructure Signed-off-by: Arjentix <arjentix@gmail.com>
675913b
to
9547d20
Compare
Description of the Change
Add new field into Domain structure. Field represents IPFS link into Domain logo.
Add
IpfsPath
structure to superficially validate pathI haven't find good IPFS paths specifications, so I was inspiring these realisations:
Issue
Resolves #1606
Benefits
Now domains can have logo
Possible Drawbacks
IPFS paths isn't fully validated. Full validation requires hash decoding and possible requesting IPFS system to check, if image really exists.
Checkout #1885
Usage Examples or Tests [optional]
See
client/integration_tests/add_domain.rs