Skip to content
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.

Sign up for GitHub

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

Refactor x/collection,token proto #3

Closed
wants to merge 5 commits into from
Closed

Refactor x/collection,token proto #3

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
10620a7
Refactor x/token proto
0Tech Dec 6, 2022
11c92c6
Refactor x/collection proto
0Tech Dec 6, 2022
f7e07c7
Modify comment
0Tech Jan 4, 2023
6e76ea7
Revert changes on token_type
0Tech Jan 4, 2023
efbbd1f
Rename a fieid to token_type
0Tech Jan 4, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 4 additions & 12 deletions proto/lbm/collection/v1/collection.proto
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,13 +17,13 @@ message Params {
// Contract defines the information of the contract for the collection.
message Contract {
// contract_id defines the unique identifier of the contract.
string contract_id = 1;
string id = 1;
// name defines the human-readable name of the contract.
string name = 2;
// meta is a brief description of the contract.
string meta = 3;
// base img uri is an uri for the contract image stored off chain.
string base_img_uri = 4;
// uri for the relevant resource stored off chain.
string uri = 4;
}

// FTClass defines the class of fungible token.
Expand Down Expand Up @@ -121,7 +121,7 @@ message TokenType {
// contract id associated with the contract.
string contract_id = 1;
// token type defines the unique identifier of the token type.
string token_type = 2;
string class_id = 2;
// name defines the human-readable name of the token type.
string name = 3;
// meta is a brief description of the token type.
Expand Down Expand Up @@ -203,11 +203,3 @@ message Attribute {
string key = 1;
string value = 2;
}

// Deprecated: use Attribute
//
// Change defines a field-value pair.
message Change {
string field = 1;
string value = 2;
}
66 changes: 33 additions & 33 deletions proto/lbm/collection/v1/event.proto
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,10 +74,10 @@ enum AttributeKey {
ATTRIBUTE_KEY_NEW_ROOT_TOKEN_ID = 19 [(gogoproto.enumvalue_customname) = "AttributeKeyNewRoot"];
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does enum AttributeKey affect legacy events?
If not, I guess the following definition should be modified into ATTRIBUTE_KEY_URI and AttributeKeyURI. (Possibly AttributeKeyUri is better?)

  ATTRIBUTE_KEY_BASE_IMG_URI      = 8 [(gogoproto.enumvalue_customname) = "AttributeKeyBaseImgURI"];

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It does. So we would better not touch it.

}

// EventSent is emitted when tokens are transferred.
// EventSend is emitted when tokens are transferred.
//
// Since: 0.46.0 (finschia)
message EventSent {
message EventSend {
// contract id associated with the contract.
string contract_id = 1;
// address which triggered the send.
Expand All @@ -90,10 +90,10 @@ message EventSent {
repeated Coin amount = 5 [(gogoproto.nullable) = false];
}

// EventAuthorizedOperator is emitted when a holder authorizes an operator to manipulate its tokens.
// EventAuthorizeOperator is emitted when a holder authorizes an operator to manipulate its tokens.
//
// Since: 0.46.0 (finschia)
message EventAuthorizedOperator {
message EventAuthorizeOperator {
// contract id associated with the contract.
string contract_id = 1;
// address of a holder which authorized the `operator` address as an operator.
Expand All @@ -102,10 +102,10 @@ message EventAuthorizedOperator {
string operator = 3;
}

// EventRevokedOperator is emitted when an authorization is revoked.
// EventRevokeOperator is emitted when an authorization is revoked.
//
// Since: 0.46.0 (finschia)
message EventRevokedOperator {
message EventRevokeOperator {
// contract id associated with the contract.
string contract_id = 1;
// address of a holder which revoked the `operator` address as an operator.
Expand All @@ -114,10 +114,10 @@ message EventRevokedOperator {
string operator = 3;
}

// EventCreatedContract is emitted when a new contract is created.
// EventCreateContract is emitted when a new contract is created.
//
// Since: 0.46.0 (finschia)
message EventCreatedContract {
message EventCreateContract {
// address which created the contract.
string creator = 1;
// contract id associated with the contract.
Expand All @@ -127,13 +127,13 @@ message EventCreatedContract {
// metadata of the contract.
string meta = 4;
// uri for the contract image stored off chain.
string base_img_uri = 5;
string uri = 5;
}

// EventCreatedFTClass is emitted when a new fungible token class is created.
// EventCreateFTClass is emitted when a new fungible token class is created.
//
// Since: 0.46.0 (finschia)
message EventCreatedFTClass {
message EventCreateFTClass {
// contract id associated with the contract.
string contract_id = 1;
// address which triggered the create.
Expand All @@ -150,10 +150,10 @@ message EventCreatedFTClass {
bool mintable = 7;
}

// EventCreatedNFTClass is emitted when a new non-fungible token class is created.
// EventCreateNFTClass is emitted when a new non-fungible token class is created.
//
// Since: 0.46.0 (finschia)
message EventCreatedNFTClass {
message EventCreateNFTClass {
// contract id associated with the contract.
string contract_id = 1;
// address which triggered the create.
Expand All @@ -166,12 +166,12 @@ message EventCreatedNFTClass {
string meta = 5;
}

// EventGranted is emitted when a granter grants its permission to a grantee.
// EventGrant is emitted when a granter grants its permission to a grantee.
//
// Info: `granter` would be empty if the permission is granted by an issuance.
//
// Since: 0.46.0 (finschia)
message EventGranted {
message EventGrant {
// contract id associated with the contract.
string contract_id = 1;
// address of the granter which grants the permission.
Expand All @@ -182,10 +182,10 @@ message EventGranted {
Permission permission = 4;
}

// EventRenounced is emitted when a grantee renounced its permission.
// EventRenounce is emitted when a grantee renounced its permission.
//
// Since: 0.46.0 (finschia)
message EventRenounced {
message EventRenounce {
// contract id associated with the contract.
string contract_id = 1;
// address of the grantee which abandons its grant.
Expand All @@ -194,10 +194,10 @@ message EventRenounced {
Permission permission = 3;
}

// EventMintedFT is emitted when fungible tokens are minted.
// EventMintFT is emitted when fungible tokens are minted.
//
// Since: 0.46.0 (finschia)
message EventMintedFT {
message EventMintFT {
// contract id associated with the contract.
string contract_id = 1;
// address which triggered the mint.
Expand All @@ -208,10 +208,10 @@ message EventMintedFT {
repeated Coin amount = 4 [(gogoproto.nullable) = false];
}

// EventMintedNFT is emitted when non-fungible tokens are minted.
// EventMintNFT is emitted when non-fungible tokens are minted.
//
// Since: 0.46.0 (finschia)
message EventMintedNFT {
message EventMintNFT {
// contract id associated with the contract.
string contract_id = 1;
// address which triggered the mint.
Expand All @@ -222,10 +222,10 @@ message EventMintedNFT {
repeated NFT tokens = 4 [(gogoproto.nullable) = false];
}

// EventBurned is emitted when tokens are burnt.
// EventBurn is emitted when tokens are burnt.
//
// Since: 0.46.0 (finschia)
message EventBurned {
message EventBurn {
// contract id associated with the contract.
string contract_id = 1;
// address which triggered the burn.
Expand All @@ -236,10 +236,10 @@ message EventBurned {
repeated Coin amount = 4 [(gogoproto.nullable) = false];
}

// EventModifiedContract is emitted when the information of a contract is modified.
// EventModifyContract is emitted when the information of a contract is modified.
//
// Since: 0.46.0 (finschia)
message EventModifiedContract {
message EventModifyContract {
// contract id associated with the contract.
string contract_id = 1;
// address which triggered the modify.
Expand All @@ -248,10 +248,10 @@ message EventModifiedContract {
repeated Attribute changes = 3 [(gogoproto.nullable) = false];
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about explaining possible keys of changes like the following?

  // changes of the attributes applied.
  // The possible attribute keys are same as the possible keys of `changes` in `message MsgModify`.
  repeated Attribute changes = 3 [(gogoproto.nullable) = false];

}

// EventModifiedTokenClass is emitted when the information of a token class is modified.
// EventModifyTokenClass is emitted when the information of a token class is modified.
//
// Since: 0.46.0 (finschia)
message EventModifiedTokenClass {
message EventModifyTokenClass {
// contract id associated with the contract.
string contract_id = 1;
// address which triggered the modify.
Expand All @@ -264,10 +264,10 @@ message EventModifiedTokenClass {
string type_name = 5;
}
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ditto, how about explaining possible keys of changes like the following?

  // changes of the attributes applied.
  // The possible attribute keys are same as the possible keys of `changes` in `message MsgModify`.
  repeated Attribute changes = 4 [(gogoproto.nullable) = false];


// EventModifiedNFT is emitted when the information of a non-fungible token is modified.
// EventModifyNFT is emitted when the information of a non-fungible token is modified.
//
// Since: 0.46.0 (finschia)
message EventModifiedNFT {
message EventModifyNFT {
// contract id associated with the contract.
string contract_id = 1;
// address which triggered the modify.
Expand All @@ -278,10 +278,10 @@ message EventModifiedNFT {
repeated Attribute changes = 4 [(gogoproto.nullable) = false];
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ditto, how about explaining possible keys of changes like the following?

  // changes of the attributes applied.
  // The possible attribute keys are same as the possible keys of `changes` in `message MsgModify`.
  repeated Attribute changes = 4 [(gogoproto.nullable) = false];

}

// EventAttached is emitted when a token is attached to another.
// EventAttach is emitted when a token is attached to another.
//
// Since: 0.46.0 (finschia)
message EventAttached {
message EventAttach {
// contract id associated with the contract.
string contract_id = 1;
// address which triggered the attach.
Expand All @@ -294,10 +294,10 @@ message EventAttached {
string target = 5;
}

// EventDetached is emitted when a token is detached from its parent.
// EventDetach is emitted when a token is detached from its parent.
//
// Since: 0.46.0 (finschia)
message EventDetached {
message EventDetach {
// contract id associated with the contract.
string contract_id = 1;
// address which triggered the detach.
Expand Down
Loading