-
Notifications
You must be signed in to change notification settings - Fork 13
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
AssetInfo implementations #22
Conversation
A New commit with review changes has been pushed |
Fix test comment Co-authored-by: piobab <[email protected]>
Thanks for update. LGTM |
@piobab Is possible to release the new version also on Crates.io? |
@Rhaki I can publish as |
@piobab LGTM |
@piobab LGTM! |
Thx for contribution! Great job! New version published in crates.io |
This pull request would add some useful implementations to
AssetInfo
:#[derive(Eq, PartialOrd, Ord, Hash)]
toAssetInfoBase<T: AddressLike>
allowing to useAssetInfoBase
as key for bothHashMap
andBTreeMap
;fn inner()
forAssetInfo
which return the innercw20 addr
ordenom
wrapped within;Prefix
andSuffix
types have been changed from()
-()
toString
-String
onPrimaryKey
implementation forAssetInfo
. This allow to iterate over aMap<AssetInfo, T>
by prefix that can becw20:
ornative:
(colon is required to match with the firstkey
generated infn key()
onPrimaryKey
implementation);