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

[LIQ-726] Edition Drop Feature #324

Merged
merged 2 commits into from
Aug 8, 2022
Merged

[LIQ-726] Edition Drop Feature #324

merged 2 commits into from
Aug 8, 2022

Conversation

broskicodes
Copy link
Contributor

No description provided.

@broskicodes broskicodes requested review from RustySol and haVincy August 3, 2022 02:40
Copy link
Contributor

@haVincy haVincy left a comment

Choose a reason for hiding this comment

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

How can we escape the dups in V1 and V2 to maintain or iterate the codes now? Any idea?

core/sdk/src/CandyShop.ts Show resolved Hide resolved
core/sdk/src/EditionDrop.ts Outdated Show resolved Hide resolved
core/sdk/src/EditionDrop.ts Outdated Show resolved Hide resolved
core/sdk/src/EditionDrop.ts Outdated Show resolved Hide resolved
core/sdk/src/EditionDrop.ts Outdated Show resolved Hide resolved
core/sdk/src/EditionDrop.ts Outdated Show resolved Hide resolved
core/sdk/src/vendor/utils/programUtils.ts Outdated Show resolved Hide resolved
@broskicodes
Copy link
Contributor Author

How can we escape the dups in V1 and V2 to maintain or iterate the codes now? Any idea?

You mean the dups between auction and marketplace v1/v2? I believe @RustySol wanted to keep them so that we can update v2 ixs without impacting v1.

@haVincy
Copy link
Contributor

haVincy commented Aug 4, 2022

You mean the dups between auction and marketplace v1/v2?

Yes... it seems we will keep growing those and need double work to maintain or fix any issue.

I believe @RustySol wanted to keep them so that we can update v2 ixs without impacting v1.

I've talked about this to RustySol last time while refactoring the structures. RustySol, do you have any idea how do we reduce the work here.

@broskicodes
Copy link
Contributor Author

Yes... it seems we will keep growing those and need double work to maintain or fix any issue.

I believe that the intent is to leave v1 as is forever now, possibly deprecate in the future (@RustySol please confirm), and only update v2 going forward

@RustySol
Copy link
Contributor

RustySol commented Aug 5, 2022

Yes... it seems we will keep growing those and need double work to maintain or fix any issue.

I believe that the intent is to leave v1 as is forever now, possibly deprecate in the future (@RustySol please confirm), and only update v2 going forward

Yes this is the plan I believe

@RustySol RustySol force-pushed the feat/edition-drop branch 3 times, most recently from 21e5fb7 to 00acb6c Compare August 8, 2022 07:04
Copy link
Contributor

@haVincy haVincy left a comment

Choose a reason for hiding this comment

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

Please help to make it a one commit with descriptive content before merge.

@RustySol
Copy link
Contributor

RustySol commented Aug 8, 2022

Edition Drop

Edition Drop feature empowers creator by selling Copies based on the Master Edition NFT.
Buyers will get a randomized Edition numbered copy upon mint.

2 Main operations included in this PR are

  • commitMasterNft: Master Edition owner will use this to transfer the Master Edition NFT into a vault to kickstart the sale of the corresponding copies.
  • mintNewPrint: Edition NFT Buyers will use this to Mint out randomized edition-ed NFTs.

@RustySol RustySol force-pushed the feat/edition-drop branch from 5df103b to 642a0ac Compare August 8, 2022 13:15
@liqnft/candy-shop-types -> 0.2.22
@liqnft/candy-shop-sdk -> 0.5.23
@RustySol RustySol merged commit ae8c8be into master Aug 8, 2022
@RustySol RustySol changed the title [LIQ-726] Add edition drop instructions to sdk [LIQ-726] Edition Drop Feature Aug 8, 2022
@haVincy haVincy deleted the feat/edition-drop branch August 9, 2022 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants