Skip to content

Bring DID/NFT effort into release/1.4.0 #11662

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
Merged
wallentx merged 187 commits into from
May 27, 2022
Merged

Bring DID/NFT effort into release/1.4.0 #11662

wallentx merged 187 commits into from
May 27, 2022

Conversation

@AmineKhaldi
Copy link
Contributor

@AmineKhaldi AmineKhaldi commented May 27, 2022

No description provided.

matt-o-how and others added 30 commits February 3, 2022 14:18
@matt-o-how
initial commit
c18fe51
@matt-o-how
assert coin added in test
509ef78
@matt-o-how
add support functions
a810bb9
@matt-o-how
add RPC calls (untestested and surely unworking)
eccb68f 
also change innerpuz
@matt-o-how
missing comma
ac4731b
@matt-o-how
update puzzle clvm hex sha256
125c27c
@Yostra @matt-o-how
fixes
f7fa15b
@matt-o-how
add load_wallet for nft_wallet
68aafea
@matt-o-how
add singleton 1.1 and update innerpuz for it
b8f5d98
@matt-o-how
update wallet for new innerpuz
8060f57
@matt-o-how
use singleton 1.1
3974f13
@matt-o-how
swap DID to singleton 1.1
0c567b5
@matt-o-how
Fix DID Wallet and DID Innerpuz
aa5e91f
@matt-o-how
list hint
2e61215
@Yostra @matt-o-how
don't push automatically
bbf374a
@matt-o-how
change to determine_coin_type
b6d4f1e
@Yostra @matt-o-how
stash debug
e246ad2
@matt-o-how
fix get_hints()
46995ea
@matt-o-how
small fix en route to bigger fix later
d736359
@matt-o-how
push for straya
d35abe3
@Yostra @matt-o-how
checkpoint
bf95259
@matt-o-how
coin discovered in wallet
6620fd9
@matt-o-how
re-enable push of attests because message spends are different
2e623fd
@matt-o-how
change a few node calls for rebase and rebase
133a607
@matt-o-how
black and flake8
a3ff74a
@matt-o-how
flake8 test formatting
794d2f2
@matt-o-how
Add transfer test and some transfer functions
48dd2dc
@matt-o-how
fix bug where innerpuzzle would not recreate itself properly
fae9f9f
@matt-o-how
fix get_info for received NFT and reovme breakpoint
f35b4ec
@matt-o-how
fix messages
359ac29
AmineKhaldi and others added 21 commits May 11, 2022 17:32
@AmineKhaldi
NFT0: Make the wallet ID required for adding a URI.
4951929
@trepca
Merge pull request Chia-Network#11485 from AmineKhaldi/nft0_list_nfts…
96f7767 
…_cmd

Initial iteration of the NFT0 chia wallet nft list command
@paninaro
Bumping GUI to 725d77a
409f5cc
@trepca
Merge pull request Chia-Network#11487 from AmineKhaldi/add_uri_wallet…
5d84814 
…_id_required

NFT0: Make the wallet ID required for adding a URI
@AmineKhaldi
NFT0: Make the wallet ID required for transferring an NFT.
cf1e5a7
@trepca
Merge pull request Chia-Network#11488 from AmineKhaldi/transfer_nft_w…
babe3ff 
…allet_id_required

NFT0: Make the wallet ID required for transferring an NFT
@ytx1991
Add fee option for NFT & limit NFT wallet creation (Chia-Network#11492)
bbcf6fd 
* Add fee option for NFT & limit NFT wallet creation

* Fix pre-commit

* Add fee for update metadata
@AmineKhaldi
Add the ability to set fees for minting NFTs, transferring NFTs and a…
102280e 
…dding URIs to NFTs.
@AmineKhaldi
Fix handling fees on the RPC API side.
50f8ab9
@trepca
Merge pull request Chia-Network#11493 from AmineKhaldi/nft0_cli_fees
15dfb0a 
Add the ability to set fees from CLI for minting NFTs, transferring NFTs and adding URIs to NFTs
@trepca
fix for missing minted coins
b2617b0
@trepca
Merge pull request Chia-Network#11500 from trepca/nft_eve_spend_hints
97e299b 
fix for missing minted coins
@trepca
updating metadata with multiple uris fix
4a38631
@trepca
Merge pull request Chia-Network#11538 from trepca/nft0_metadata_bug
5128d50 
updating metadata with multiple uris fix
@xdustinface
Merge remote-tracking branch 'chia/main' into pr-main-dids-catchup
3ee5504
@AmineKhaldi
Merge main into pr-main-dids-catchup
a9b1ba8
@AmineKhaldi
Consolidate test_did_rpc.py into test_wallet_rpc.py and make the test…
dc804d9 
…s use the wallet RPC client. (#4)
@AmineKhaldi
Update workflows and minor formatting.
28d3e77
@trepca @AmineKhaldi
faster tests, other fixes
16cb1c3
@trepca @AmineKhaldi
potentially fixes the did test failure
0b5c4a4
@AmineKhaldi
Bring DID/NFT effort into release/1.4.0
66cbd09
@lgtm-com
Copy link

lgtm-com bot commented May 27, 2022

This pull request introduces 1 alert and fixes 1 when merging 66cbd09 into 954642a - view on LGTM.com

new alerts:

  • 1 for Unused local variable

fixed alerts:

  • 1 for Variable defined multiple times

@AmineKhaldi AmineKhaldi requested review from paninaro and trepca May 27, 2022 13:23
@wallentx wallentx requested review from cmmarslender and removed request for cmmarslender and trepca May 27, 2022 17:28
wjblanke
wjblanke approved these changes May 27, 2022
View reviewed changes
Copy link
Contributor

@wjblanke wjblanke left a comment

Choose a reason for hiding this comment

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

aok

@wallentx wallentx merged commit d0de803 into Chia-Network:release/1.4.0 May 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wjblanke wjblanke wjblanke approved these changes

+1 more reviewer

@paninaro paninaro paninaro approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.