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

Jump to bottom

NFI Pallet #866

Merged

JasonTulp merged 17 commits into main from feat/add-nfi-pallet

Aug 25, 2024

Contributor

JasonTulp commented Jul 24, 2024 •

edited

Loading

Adds the NFI pallet. This pallet is responsible for storing the mappings between NFT and SFT tokenIds and an IPFS link which points to the AI related data such as murmur matrix and Genome matrix

https://futureverse.atlassian.net/browse/TRN-466?atlOrigin=eyJpIjoiNWY4OGE5ZGJiNzMwNDY5ZDlkZDVkYTg0MzA5ZjM2YTYiLCJwIjoiaiJ9

JasonTulp added 9 commits

July 17, 2024 13:04


          Add NFI pallet + some tests

a491958


          Add extra unit tests

c042119


          Add benchmarks and temp weights

5d8e92f


          include weights in lib

1f2ecfc


          Merge remote-tracking branch 'origin/main' into feat/add-nfi-pallet

3b7db6d

# Conflicts:
#	pallet/sft/src/lib.rs
#	runtime/Cargo.toml


          Rebase and add weights

747a01b


          Merge remote-tracking branch 'origin/main' into feat/add-nfi-pallet

e353a51


          Fix build errors

e26bf90

fmt

1fe853c

JasonTulp requested review from surangap and zees-dev and removed request for surangap

August 15, 2024 02:18

JasonTulp marked this pull request as ready for review

August 15, 2024 02:18

JasonTulp requested a review from surangap

August 15, 2024 02:19


          Remove Jen subtype

0f3ba09

zees-dev reviewed

View reviewed changes

pallet/common/src/test_utils.rs Outdated Show resolved Hide resolved

zees-dev reviewed

View reviewed changes

pallet/nfi/src/lib.rs Outdated Show resolved Hide resolved

zees-dev reviewed

View reviewed changes

pallet/nfi/src/lib.rs Show resolved Hide resolved

zees-dev reviewed

View reviewed changes

pallet/nfi/src/lib.rs Outdated Show resolved Hide resolved

zees-dev reviewed

View reviewed changes

pallet/nfi/src/lib.rs Show resolved Hide resolved

zees-dev reviewed

View reviewed changes

pallet/nft/src/lib.rs Show resolved Hide resolved

zees-dev reviewed

View reviewed changes

pallet/sft/src/lib.rs Show resolved Hide resolved

zees-dev reviewed

View reviewed changes

pallet/sft/src/mock.rs Outdated Show resolved Hide resolved

JasonTulp added 3 commits

August 21, 2024 10:47


          Address PR comments

e5e9de4


          Merge remote-tracking branch 'origin/main' into feat/add-nfi-pallet

47cccd0


          Add burn handling + permissions for collection owner to manually requ…

979c6ac

…est data

surangap reviewed

View reviewed changes

pallet/nfi/Cargo.toml Outdated Show resolved Hide resolved

surangap reviewed

View reviewed changes

pallet/nfi/src/lib.rs Outdated Show resolved Hide resolved

surangap reviewed

View reviewed changes

pallet/nfi/src/lib.rs Show resolved Hide resolved

surangap reviewed

View reviewed changes

Contributor

surangap left a comment

overall lgtm, nice work. Added a couple of comments.

JasonTulp added 3 commits

August 22, 2024 10:30


          Update comments and naming for check_permissions function

7e8b377


          Update comments and naming for check_permissions function + add data …

17c4271

…removed event


          Update dependencies to workspace deps

f0b899c

surangap approved these changes

View reviewed changes

Contributor

surangap left a comment

Do a CI benchmark run as well.

zees-dev approved these changes

View reviewed changes

Contributor

zees-dev left a comment

LGTM 👍


          Update benchmarks for pallet-nfi on feat/add-nfi-pallet

334858d

JasonTulp merged commit b4b578c into main

JasonTulp deleted the feat/add-nfi-pallet branch

August 25, 2024 07:33

JasonTulp mentioned this pull request

Release/8.57.0 #874

Merged

4 tasks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet