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: Delegatable Utility Tokens Derived from NFTs #6884

Closed
wants to merge 12 commits into from

Conversation

blgaman
Copy link

@blgaman blgaman commented Apr 16, 2023

No description provided.

@blgaman blgaman requested a review from eth-bot as a code owner April 16, 2023 00:11
@eth-bot
Copy link
Collaborator

eth-bot commented Apr 16, 2023

File EIPS/eip-6884.md

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

@eth-bot eth-bot added e-consensus Waiting on editor consensus e-review Waiting on editor to review labels Apr 16, 2023
@eth-bot eth-bot changed the base branch from master to eipbot/6884 April 16, 2023 00:12
@eth-bot eth-bot changed the base branch from eipbot/6884 to master April 16, 2023 00:12
@github-actions github-actions bot added the w-ci Waiting on CI to pass label Apr 16, 2023
@SamWilsn SamWilsn closed this Apr 16, 2023
@SamWilsn SamWilsn reopened this Apr 16, 2023
@github-actions github-actions bot added c-new Creates a brand new proposal s-draft This EIP is a Draft t-erc labels Apr 26, 2023
@eth-bot eth-bot changed the title Add EIP: Extendable Utility Tokens Derived from Origin NFTs Add EIP: Delegatable Utility Tokens Derived from Origin NFTs Apr 26, 2023
@github-actions github-actions bot added w-ci Waiting on CI to pass and removed w-ci Waiting on CI to pass labels Apr 26, 2023
@eth-bot eth-bot changed the title Add EIP: Delegatable Utility Tokens Derived from Origin NFTs Add EIP: Delegatable Utility Tokens Derived from NFTs Apr 29, 2023
@github-actions github-actions bot added w-ci Waiting on CI to pass and removed w-ci Waiting on CI to pass labels Apr 29, 2023
@github-actions github-actions bot added w-ci Waiting on CI to pass and removed w-ci Waiting on CI to pass labels Apr 29, 2023
@github-actions
Copy link

There has been no activity on this pull request for 2 weeks. It will be closed after 3 months of inactivity. If you would like to move this PR forward, please respond to any outstanding feedback or add a comment indicating that you have addressed all required feedback and are ready for a review.

@github-actions github-actions bot added the w-stale Waiting on activity label Sep 18, 2023
@SamWilsn SamWilsn closed this Sep 18, 2023
@SamWilsn SamWilsn reopened this Sep 18, 2023
@github-actions
Copy link

The commit 08ee53d (as a parent of f702091) contains errors.
Please inspect the Run Summary for details.

eip: 6884
title: Delegatable Utility Tokens Derived from NFTs
description: Structure and interface for Delegatable Utility Tokens (DUT) for enhancing the utility of NFTs.
author: JB Won (@JB_theRedDuck), dagarn (@0xdagarn), Jennifer Lee (@purp-lee), Bora Lee (@Violet-Bora-Lee), Taehyeong Noh (@fin)
Copy link
Contributor

Choose a reason for hiding this comment

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

The first username here isn't a valid GitHub identifier.

Suggested change
author: JB Won (@JB_theRedDuck), dagarn (@0xdagarn), Jennifer Lee (@purp-lee), Bora Lee (@Violet-Bora-Lee), Taehyeong Noh (@fin)
author: JB Won, dagarn (@0xdagarn), Jennifer Lee (@purp-lee), Bora Lee (@Violet-Bora-Lee), Taehyeong Noh (@fin)


The key words “MUST”, “MUST NOT”, “REQUIRED”, “SHALL”, “SHALL NOT”, “SHOULD”, “SHOULD NOT”, “RECOMMENDED”, “MAY”, and “OPTIONAL” in this document are to be interpreted as described in RFC 2119.

**Every ERC-6884 compliant contract must implement the `ERC6884` and `ERC165` interfaces**:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
**Every ERC-6884 compliant contract must implement the `ERC6884` and `ERC165` interfaces**:
**Every ERC-6884 compliant contract must implement the `IERC6884` and `ERC165` interfaces**:


**Every ERC-6884 compliant contract must implement the `ERC6884` and `ERC165` interfaces**:

```solidity=
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
```solidity=
```solidity

```solidity=
pragma solidity ^0.8.0;

interface IERC6884 /* is IERC165 */ {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
interface IERC6884 /* is IERC165 */ {
interface IERC6884 /* is ERC165 */ {

Comment on lines +147 to +156

interface ERC165 {
/// @notice Query if a contract implements an interface
/// @param interfaceID The interface identifier, as specified in ERC-165
/// @dev Interface identification is specified in ERC-165. This function
/// uses less than 30,000 gas.
/// @return `true` if the contract implements `interfaceID` and
/// `interfaceID` is not 0xffffffff, `false` otherwise
function supportsInterface(bytes4 interfaceID) external view returns (bool);
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
interface ERC165 {
/// @notice Query if a contract implements an interface
/// @param interfaceID The interface identifier, as specified in ERC-165
/// @dev Interface identification is specified in ERC-165. This function
/// uses less than 30,000 gas.
/// @return `true` if the contract implements `interfaceID` and
/// `interfaceID` is not 0xffffffff, `false` otherwise
function supportsInterface(bytes4 interfaceID) external view returns (bool);
}

No need to duplicate the interface here. Readers can find it in EIP-165.

type: Standards Track
category: ERC
created: 2023-04-16
---
Copy link
Contributor

Choose a reason for hiding this comment

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

ERC-165 is required because you use its interface, and 721 because you use its metadata.

Suggested change
---
requires: 165, 721
---


## Rationale

NFTs are valuable assets, and moving them can be very risky. It's also risky to try to increase their utility by locking them to a specific contract. We've come up with a simple way to enhance utility without moving NFTs.
Copy link
Contributor

Choose a reason for hiding this comment

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

This paragraph belongs in the Motivation section. The Rationale section should justify technical decisions made within the EIP itself, while the Motivation should justify the EIP as a whole.


There might be dApps build based on supply and demand for usage rights, facilitated by this delegation process.

## **Test Cases**
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
## **Test Cases**
## Test Cases

➜ yarn hardhat test test/test.js
```

## **Reference Implementation**
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
## **Reference Implementation**
## Reference Implementation

Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove build system files.

@github-actions github-actions bot removed the w-stale Waiting on activity label Sep 19, 2023
@github-actions
Copy link

github-actions bot commented Oct 9, 2023

There has been no activity on this pull request for 2 weeks. It will be closed after 3 months of inactivity. If you would like to move this PR forward, please respond to any outstanding feedback or add a comment indicating that you have addressed all required feedback and are ready for a review.

@github-actions github-actions bot added the w-stale Waiting on activity label Oct 9, 2023
@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.

@SamWilsn SamWilsn closed this Oct 25, 2023
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 w-ci Waiting on CI to pass w-stale Waiting on activity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants