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

Propose Emotable Extension for Non-Fungible Tokens #6381

Merged
merged 6 commits into from
Feb 7, 2023

Conversation

ThunderDeliverer
Copy link
Contributor

This EIP draft proposal includes RMRK team's work on developing an NFT standard where an NFT can be reacted to using Unicode emojis.

Proposed an interface for Non-Fungible Tokens extension allowing for
reacting to them using Unicode emojis.
@github-actions github-actions bot added c-new Creates a brand new proposal e-number Waiting on EIP Number assignment s-draft This EIP is a Draft t-erc labels Jan 26, 2023
@github-actions
Copy link

The commit 85ea5f9 (as a parent of 31a064c) contains errors.
Please inspect the Run Summary for details.

@github-actions github-actions bot added the w-ci Waiting on CI to pass label Jan 26, 2023
@eth-bot
Copy link
Collaborator

eth-bot commented Jan 26, 2023

All tests passed; auto-merging...

(pass) eip-6381.md

classification
updateEIP
  • passed!

(pass) assets/eip-6381/contracts/Emotable.sol

classification
ambiguous
  • file assets/eip-6381/contracts/Emotable.sol is associated with EIP 6381; because there are also changes being made to EIPS/eip-6381.md all changes to corresponding assets are also allowed

(pass) assets/eip-6381/contracts/IEmotable.sol

classification
ambiguous
  • file assets/eip-6381/contracts/IEmotable.sol is associated with EIP 6381; because there are also changes being made to EIPS/eip-6381.md all changes to corresponding assets are also allowed

(pass) assets/eip-6381/contracts/mocks/ERC721EmotableMock.sol

classification
ambiguous
  • file assets/eip-6381/contracts/mocks/ERC721EmotableMock.sol is associated with EIP 6381; because there are also changes being made to EIPS/eip-6381.md all changes to corresponding assets are also allowed

(pass) assets/eip-6381/hardhat.config.ts

classification
ambiguous
  • file assets/eip-6381/hardhat.config.ts is associated with EIP 6381; because there are also changes being made to EIPS/eip-6381.md all changes to corresponding assets are also allowed

(pass) assets/eip-6381/package.json

classification
ambiguous
  • file assets/eip-6381/package.json is associated with EIP 6381; because there are also changes being made to EIPS/eip-6381.md all changes to corresponding assets are also allowed

(pass) assets/eip-6381/test/emotable.ts

classification
ambiguous
  • file assets/eip-6381/test/emotable.ts is associated with EIP 6381; because there are also changes being made to EIPS/eip-6381.md all changes to corresponding assets are also allowed

The proposal number was assigned based on the PR number. If the editors
desire to change the PR number, that can be easily arranged.
@github-actions github-actions bot removed the e-number Waiting on EIP Number assignment label Jan 26, 2023
@github-actions github-actions bot removed the w-ci Waiting on CI to pass label Jan 26, 2023
EIPS/eip-6381.md Outdated

```solidity
/// @title EIP-6381 Emotable Extension for Non-Fungible Tokens
/// @dev See https://eips.ethereum.org/EIPS/eip-5773
Copy link
Contributor

Choose a reason for hiding this comment

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

Wrong EIP number?

Copy link
Contributor

Choose a reason for hiding this comment

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

Fixed

EIPS/eip-6381.md Outdated

pragma solidity ^0.8.16;

interface IRMRKEmotable 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.

Dosen't match the interface in the reference.

Copy link
Contributor

Choose a reason for hiding this comment

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

Fixed

steven2308 and others added 2 commits February 7, 2023 10:30
Updates description of EIP

Co-authored-by: Sam Wilson <[email protected]>
@eth-bot eth-bot enabled auto-merge (squash) February 7, 2023 15:36
@eth-bot eth-bot merged commit 7538bf2 into ethereum:master Feb 7, 2023
iseriohn pushed a commit to iseriohn/EIP-NFT-Rights-Management that referenced this pull request Feb 16, 2023
* Propose Emotable Extension for Non-Fungible Tokens

Proposed an interface for Non-Fungible Tokens extension allowing for
reacting to them using Unicode emojis.

* Assign a proposal number

The proposal number was assigned based on the PR number. If the editors
desire to change the PR number, that can be easily arranged.

* Add discussion URL

* Fix link typo

* Update EIPS/eip-6381.md

Updates description of EIP

Co-authored-by: Sam Wilson <[email protected]>

* Addresses editor suggestions.

---------

Co-authored-by: Steven Pineda <[email protected]>
Co-authored-by: Sam Wilson <[email protected]>
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 s-draft This EIP is a Draft t-erc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants