Skip to content

reworks some nft operations into trait #28

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

Merged
ilionic merged 24 commits into from
Dec 28, 2021
Merged

reworks some nft operations into trait #28

Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
ee77664
convert initial nft to trait w mint_nft
bmacer Dec 17, 2021
3c8b113
Merge branch 'main' into rework/rmrk-as-trait
bmacer Dec 17, 2021
b697e4a
convert burn-nft into trait
bmacer Dec 17, 2021
162b1fa
convert send to trait
bmacer Dec 17, 2021
587062d
change permill to default
bmacer Dec 17, 2021
c8a23b2
Merge branch 'main' into rework/rmrk-as-trait
bmacer Dec 20, 2021
aae468f
merge main
bmacer Dec 20, 2021
c877dc0
cleaning up
bmacer Dec 20, 2021
d768b3d
clean up
bmacer Dec 20, 2021
8dcfc0d
re-add uniques collection and nft ops
bmacer Dec 20, 2021
012dd72
move impl to functions.rs
bmacer Dec 20, 2021
3d585d9
rename maxnftrecurrsions to maxrecursions
bmacer Dec 20, 2021
991873f
NftId and CollectionId should be concrete types
ilionic Dec 21, 2021
a5c22e1
remove NftId and CollectionId from generics
ilionic Dec 21, 2021
455e46a
refactor Result import
ilionic Dec 21, 2021
8b44e06
CR comments (is_none tests)
ilionic Dec 21, 2021
4c1a84f
simplify self function calls
ilionic Dec 21, 2021
9760209
CR comment
ilionic Dec 21, 2021
6335693
remove seialize derive
bmacer Dec 23, 2021
c09baf7
An insert override the remove operation, no need an additional remove
ilionic Dec 28, 2021
0b9e103
change sender clone to reference
ilionic Dec 28, 2021
9f0f17d
CR comments, improve readability
ilionic Dec 28, 2021
83eb5bf
Merge branch 'rework/rmrk-as-trait' of github.com:rmrk-team/rmrk-subs…
ilionic Dec 28, 2021
1c2dc49
cleanup uniques integration
ilionic Dec 28, 2021
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
Prev Previous commit
Next Next commit
change sender clone to reference 
Co-authored-by: h4x3rotab <[email protected]>
  • Loading branch information
@ilionic @h4x3rotab
ilionic and h4x3rotab authored Dec 28, 2021
commit 0b9e103fdaa8a03c3f59e879bb39f8481925de81
2 changes: 1 addition & 1 deletion pallets/rmrk-core/src/functions.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ impl<T: Config> Nft<T::AccountId, StringLimitOf<T>> for Pallet<T> {
) -> sp_std::result::Result<(CollectionId, NftId), DispatchError> {
let mut sending_nft =
NFTs::<T>::get(collection_id, nft_id).ok_or(Error::<T>::NoAvailableNftId)?;
ensure!(sending_nft.rootowner == sender.clone(), Error::<T>::NoPermission);
ensure!(&sending_nft.rootowner == &sender, Error::<T>::NoPermission);

match new_owner.clone() {
AccountIdOrCollectionNftTuple::AccountId(account_id) => {
Expand Down