-
Notifications
You must be signed in to change notification settings - Fork 6
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
esdts-improvements #270
esdts-improvements #270
Conversation
core/constants.go
Outdated
// NonFungibleV2 defines the token type for ESDT non fungible tokens | ||
NonFungibleV2 | ||
// Meta defines the token type for ESDT meta tokens | ||
Meta |
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.
We have SFT as well. Plus bring here the DynamicNFT, DynamicSFT, DynamicMeta as well.
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.
Added.
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.
change the order of semiFungible and Meta - maybe add MetaFungible - to have a complete name.
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.
+1
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.
Done.
return uint32(NonFungible), nil | ||
case NonFungibleESDTv2: | ||
return uint32(NonFungibleV2), nil | ||
case MetaESDT: |
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.
add the more types, dynamics as well.
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.
Done.
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.
MetaFungibleESDT
…n-constants # Conflicts: # data/outport/common_test.go
@@ -177,3 +177,32 @@ func ConvertToEvenHexBigInt(value *big.Int) string { | |||
|
|||
return str | |||
} | |||
|
|||
// ConvertESDTTypeToUint32 converts the esdt type to uint32 | |||
func ConvertESDTTypeToUint32(esdtType string) (uint32, error) { |
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.
tests?
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.
Added.
core/constants.go
Outdated
// NonFungibleV2 defines the token type for ESDT non fungible tokens | ||
NonFungibleV2 | ||
// Meta defines the token type for ESDT meta tokens | ||
Meta |
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.
+1
No description provided.