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 ERC: NFT Dynamic Traits #26

Merged
merged 18 commits into from
Nov 29, 2023

Conversation

ryanio
Copy link
Contributor

@ryanio ryanio commented Oct 26, 2023

@eip-review-bot
Copy link
Collaborator

eip-review-bot commented Oct 31, 2023

✅ All reviewers have approved.

@ryanio ryanio changed the title Add ERC: NFT Dynamic Traits Add ERC-7496: NFT Dynamic Traits Oct 31, 2023
@g11tech g11tech force-pushed the ryan/add-eip-nft-dynamic-traits branch from 06f29e1 to 041d584 Compare November 3, 2023 15:25
@eip-review-bot eip-review-bot changed the title Add ERC-7496: NFT Dynamic Traits Add ERC: NFT Dynamic Traits Nov 3, 2023
@github-actions github-actions bot added w-ci and removed w-ci labels Nov 3, 2023
@github-actions github-actions bot removed the w-ci label Nov 3, 2023
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.

approving the PR as all the comments from the parent EIP PR seemed to be addressed :

ethereum/EIPs#7500 cc @SamWilsn

@g11tech g11tech enabled auto-merge (squash) November 3, 2023 20:02
g11tech
g11tech previously approved these changes Nov 3, 2023
- use 0x000...000 instead of 0x0
- add min/maxValue for decimal
- update impl to latest
@SamWilsn SamWilsn closed this Nov 28, 2023
auto-merge was automatically disabled November 28, 2023 15:17

Pull request was closed

@SamWilsn SamWilsn reopened this Nov 28, 2023
@eip-review-bot eip-review-bot enabled auto-merge (squash) November 28, 2023 15:18
eip-review-bot
eip-review-bot previously approved these changes Nov 28, 2023
Copy link
Collaborator

@eip-review-bot eip-review-bot left a comment

Choose a reason for hiding this comment

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

All Reviewers Have Approved; Performing Automatic Merge...

@ryanio
Copy link
Contributor Author

ryanio commented Nov 28, 2023

test comment to trigger Auto Review Bot Trigger / Trigger

@SamWilsn
Copy link
Collaborator

Might be stuck because of your assets directory. I believe it's supposed to be ../assets/erc-7496.

auto-merge was automatically disabled November 28, 2023 15:47

Head branch was pushed to by a user without write access

@ryanio ryanio dismissed stale reviews from eip-review-bot and SamWilsn via e578575 November 28, 2023 15:47
@github-actions github-actions bot added the w-ci label Nov 28, 2023
Copy link

The commit 1c77ed7 (as a parent of 736d57c) contains errors.
Please inspect the Run Summary for details.

@github-actions github-actions bot removed the w-ci label Nov 28, 2023
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.

approving following @SamWilsn 's previous approval prior to merging woes

@SamWilsn SamWilsn closed this Nov 28, 2023
@SamWilsn SamWilsn reopened this Nov 28, 2023
@eip-review-bot eip-review-bot enabled auto-merge (squash) November 28, 2023 18:48
Copy link
Collaborator

@eip-review-bot eip-review-bot left a comment

Choose a reason for hiding this comment

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

All Reviewers Have Approved; Performing Automatic Merge...

Copy link
Collaborator

@eip-review-bot eip-review-bot left a comment

Choose a reason for hiding this comment

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

All Reviewers Have Approved; Performing Automatic Merge...

@Pandapip1 Pandapip1 disabled auto-merge November 28, 2023 19:26
@Pandapip1 Pandapip1 enabled auto-merge (squash) November 28, 2023 19:26
auto-merge was automatically disabled November 29, 2023 17:11

Base branch was modified

@Pandapip1 Pandapip1 merged commit b51a744 into ethereum:master Nov 29, 2023
19 of 21 checks passed
@ryanio ryanio deleted the ryan/add-eip-nft-dynamic-traits branch April 6, 2024 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants