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

Add EIP: NFT Redeemables #7501

Closed
wants to merge 32 commits into from

Conversation

ryanio
Copy link
Contributor

@ryanio ryanio commented Aug 18, 2023

@ryanio ryanio requested a review from eth-bot as a code owner August 18, 2023 18:14
@github-actions github-actions bot added c-new Creates a brand new proposal s-draft This EIP is a Draft t-erc labels Aug 18, 2023
@eth-bot
Copy link
Collaborator

eth-bot commented Aug 18, 2023

File EIPS/eip-7498.md

Requires 1 more reviewers from @axic, @Pandapip1, @SamWilsn, @xinbenlv

@eth-bot eth-bot changed the title Add EIP: NFT Redeemables Add EIP: NFT Dynamic Traits Aug 18, 2023
@eth-bot eth-bot added e-consensus Waiting on editor consensus e-review Waiting on editor to review labels Aug 18, 2023
@github-actions github-actions bot added the w-ci Waiting on CI to pass label Aug 18, 2023
@ryanio ryanio changed the title Add EIP: NFT Dynamic Traits Add EIP: NFT Redeemables Aug 18, 2023
@eth-bot eth-bot changed the title Add EIP: NFT Redeemables Add EIP: NFT Dynamic Traits Aug 18, 2023
@github-actions github-actions bot removed the w-ci Waiting on CI to pass label Aug 18, 2023
@github-actions github-actions bot added the w-ci Waiting on CI to pass label Aug 18, 2023
@eth-bot eth-bot changed the title Add EIP: NFT Dynamic Traits Add EIP: NFT Redeemables Aug 18, 2023
@github-actions github-actions bot removed the w-ci Waiting on CI to pass label Aug 18, 2023
@github-actions github-actions bot added the w-ci Waiting on CI to pass label Aug 18, 2023
@abcoathup
Copy link
Contributor

abcoathup commented Aug 18, 2023

EIP/ERC numbering changes to sequential from 7500 and is no longer the PR number: #6990 (comment)

This PR was opened 1 minute after #7500 which was opened 3 minutes after #7499 so has the appearance of number sniping but as there is a spec I am not manually renumbering.

@github-actions github-actions bot removed the w-ci Waiting on CI to pass label Aug 19, 2023
- Add new `campaignRequirements` to allow for OR case redemptions (e.g. token from contract A OR contract B)
- Include reference implementation and tests (to be continued to be added to)
- Destructure extraData in redeem() func into params for easier serialization and understandability of params
- Simplify IRedemptionMintable and add campaignId
- Add `translations` to redeemable URI
@ryanio ryanio force-pushed the ryan/add-eip-nft-redeemables branch from efe2318 to b3f7c97 Compare September 26, 2023 01:02
@github-actions github-actions bot added the w-ci Waiting on CI to pass label Sep 26, 2023
@github-actions
Copy link

github-actions bot commented Oct 6, 2023

The commit 39e4b98 (as a parent of 9f653b8) contains errors.
Please inspect the Run Summary for details.

@github-actions github-actions bot removed the w-ci Waiting on CI to pass label Oct 12, 2023
- discovery: events and getters that provide information about the requirements of a redemption campaign
- onchain: token mints with context of items spent
- offchain: the ability to associate with ecommerce orders (through `redemptionHash`)
- trait redemptions: improving the burn-to-redeem experience with E RC-7496 <!-- [E RC-7496] (./e ip-7496.md) --> Dynamic Traits.
Copy link
Contributor

Choose a reason for hiding this comment

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

i am assuming you are waiting for 7496 draft merge to fix this link?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes exactly, otherwise CI wasn't happy, can fix as soon as #7500 is merged and I can link properly

Copy link
Contributor

@g11tech g11tech left a comment

Choose a reason for hiding this comment

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

looks good, minor comment

@SamWilsn
Copy link
Contributor

I am closing this pull request because we are in the process of separating EIPs and ERCs into distinct repositories. Unfortunately, as far as we are aware, GitHub does not provide any tools to ease this migration, so every pull request will need to be re-opened manually.

As this is a PR to create / modify an ERC, I will kindly ask you to redirect this to the new repository at ethereum/ERCs. We have prepared a guide to help with the process.

If there is relevant history here, please link to this PR from the new pull request.

On behalf of the EIP Editors, I apologize for this inconvenience.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c-new Creates a brand new proposal e-consensus Waiting on editor consensus e-review Waiting on editor to review s-draft This EIP is a Draft t-erc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants