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.

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(HIPs-646/657/765): Added implementations for metadata fields #678

Merged

Conversation

gsstoykov
Copy link
Contributor

Description:
This PR introduces metadata and metadataKey fields for token related functionalities described in HIPs-646/657/765

Related issue(s):
#445
#674

Fixes #445 #674 #443

Notes for reviewer:
Still not tested as the corresponding functionality in local-node is not yet set up. The related HIPs are closely tied in their implementations and trigger changes in common files so I am creating one PR for them all.

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
@gsstoykov gsstoykov self-assigned this Mar 22, 2024
@gsstoykov gsstoykov linked an issue Mar 22, 2024 that may be closed by this pull request
14 tasks
src/sdk/main/include/TokenCreateTransaction.h Outdated Show resolved Hide resolved
src/sdk/main/include/TokenInfo.h Outdated Show resolved Hide resolved
src/sdk/main/src/Status.cc Outdated Show resolved Hide resolved
src/sdk/main/src/TokenInfo.cc Show resolved Hide resolved
src/sdk/main/src/TokenUpdateNftsTransaction.cc Outdated Show resolved Hide resolved
src/sdk/main/src/TokenUpdateNftsTransaction.cc Outdated Show resolved Hide resolved
src/sdk/main/src/TokenUpdateNftsTransaction.cc Outdated Show resolved Hide resolved
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
@Petyo-Lukanov Petyo-Lukanov linked an issue Apr 24, 2024 that may be closed by this pull request
@gsstoykov gsstoykov marked this pull request as ready for review April 24, 2024 09:44
@gsstoykov gsstoykov requested review from a team as code owners April 24, 2024 09:44
rwalworth
rwalworth previously approved these changes Apr 24, 2024
Copy link
Contributor

@rwalworth rwalworth left a comment

Choose a reason for hiding this comment

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

LGTM just a few small things here and there

petreze
petreze previously approved these changes Apr 24, 2024
@gsstoykov gsstoykov dismissed stale reviews from petreze and rwalworth via ba37e89 April 25, 2024 08:27
gsstoykov and others added 6 commits April 25, 2024 11:27
…ts.cc

Co-authored-by: Rob Walworth <110835868+rwalworth@users.noreply.github.com>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
…ts.cc

Co-authored-by: Rob Walworth <110835868+rwalworth@users.noreply.github.com>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
…ts.cc

Co-authored-by: Rob Walworth <110835868+rwalworth@users.noreply.github.com>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
…ts.cc

Co-authored-by: Rob Walworth <110835868+rwalworth@users.noreply.github.com>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
…ts.cc

Co-authored-by: Rob Walworth <110835868+rwalworth@users.noreply.github.com>
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
@gsstoykov gsstoykov requested a review from a team as a code owner April 25, 2024 08:32
Signed-off-by: gsstoykov <georgi.stoykov@limechain.tech>
Copy link
Contributor

@rwalworth rwalworth left a comment

Choose a reason for hiding this comment

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

LGTM

@gsstoykov gsstoykov merged commit 4636d7a into main Apr 26, 2024
2 checks passed
@gsstoykov gsstoykov deleted the 00443-add-hip-657-mutable-metadata-fields-for-dynamic-nfts branch April 26, 2024 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants