-
Notifications
You must be signed in to change notification settings - Fork 19
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
feat: skeleton for x/datapool
and proto files
#283
Conversation
…3/skeleton # Conflicts: # go.mod # go.sum
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.
LGTM 👍
proto/panacea/datapool/v2/pool.proto
Outdated
uint64 target_num_data = 4; // target number of data | ||
uint64 cur_num_data = 5; // current number of data | ||
uint64 max_nft_supply = 6; // max supply of data access NFT of the pool | ||
uint64 issued_nft = 7; // current issued NFT |
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.
Does this issued_nft
means the curation NFT ID? or the number of NFTs issued?
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.
The latter. It would be better to use num_issued_nft
to make clearer. Thanks.
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.
thank you
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.
LGTM 👍
x/datapool
have been made via starport.