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

Update ERC-7007: Move to Review #209

Merged
merged 24 commits into from
Mar 5, 2024
Merged

Update ERC-7007: Move to Review #209

merged 24 commits into from
Mar 5, 2024

Conversation

socathie
Copy link
Contributor

We have updated our interface to support both validity (zkML) and fraud (opML) proofs. Added Rationale section. Also incorporated comments from Ethereum Magicians into Security Considerations.

tina1998612 and others added 12 commits January 10, 2024 20:38
* Update erc-7007.md to add opML compatibility

* Update erc-7007.md param name

* Create IOpmlLib.sol

* Update and rename ERC7007.sol to ERC7007_zkml.sol

* Update IOpmlLib.sol

* Update IOpmlLib.sol

* Update IOpmlLib.sol

* Create ERC7007_opml.sol

* Update ERC7007_opml.sol

* Update IOpmlLib.sol
* misc: improve abstract & motivation

* fix: Spec intro & Model Publication wording

* fix: Rewrite part of motivation

* corrected some small typos
* rewrote the last two paragraphs in Motivation

---------

Co-authored-by: drCathieSo.eth <socathie@users.noreply.github.com>
@eip-review-bot
Copy link
Collaborator

eip-review-bot commented Jan 18, 2024

✅ All reviewers have approved.

@eip-review-bot eip-review-bot changed the title Update ERC-7007: Move to Review Website: Move to Review Jan 18, 2024
@github-actions github-actions bot added the w-ci label Jan 18, 2024
@socathie socathie changed the title Website: Move to Review Update ERC-7007: Move to Review Jan 18, 2024
@github-actions github-actions bot added w-ci and removed w-ci labels Jan 18, 2024
@eip-review-bot eip-review-bot changed the title Update ERC-7007: Move to Review Website: Move to Review Jan 18, 2024
@github-actions github-actions bot removed the w-ci label Jan 18, 2024
@socathie socathie changed the title Website: Move to Review Update ERC-7007: Move to Review Jan 18, 2024
@eip-review-bot eip-review-bot changed the title Update ERC-7007: Move to Review Website: Move to Review Jan 20, 2024
* Update erc-7007.md: Fix wordings

* update license

* Update description and motivation -> rationale

---------

Co-authored-by: drCathieSo.eth <socathie@users.noreply.github.com>
@socathie socathie changed the title Website: Move to Review Update ERC-7007: Move to Review Feb 8, 2024
* Add workflow diagram

* Add workflow description
@eip-review-bot eip-review-bot changed the title Update ERC-7007: Move to Review Website: Move to Review Feb 8, 2024
@socathie socathie changed the title Website: Move to Review Update ERC-7007: Move to Review Feb 8, 2024
ERCS/erc-7007.md Outdated Show resolved Hide resolved
ERCS/erc-7007.md Outdated Show resolved Hide resolved
ERCS/erc-7007.md Outdated Show resolved Hide resolved
ERCS/erc-7007.md Outdated Show resolved Hide resolved
ERCS/erc-7007.md Outdated Show resolved Hide resolved
ERCS/erc-7007.md Outdated Show resolved Hide resolved
@eip-review-bot eip-review-bot changed the title Update ERC-7007: Move to Review Website: Move to Review Feb 20, 2024
Copy link

The commit 45fb7bf (as a parent of 46c3cea) contains errors.
Please inspect the Run Summary for details.

@github-actions github-actions bot added the w-ci label Feb 20, 2024
@github-actions github-actions bot removed the w-ci label Feb 20, 2024
@socathie socathie changed the title Website: Move to Review ERC-7007: Move to Review Feb 20, 2024
@eip-review-bot eip-review-bot changed the title ERC-7007: Move to Review Website: Move to Review Mar 5, 2024
@eip-review-bot eip-review-bot enabled auto-merge (squash) March 5, 2024 16:10
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...

@socathie socathie changed the title Website: Move to Review Update ERC-7007: Move to Review Mar 5, 2024
@SamWilsn SamWilsn disabled auto-merge March 5, 2024 16:44
@SamWilsn SamWilsn merged commit 479ad98 into ethereum:master Mar 5, 2024
15 of 16 checks passed
@Webbdlee74
Copy link

This was air-dropped in my Coinbase wallet during onchainsummer 2023 as an ERC-20 token in the gnosis network. The status of the token or nft at the time was undeveloped. It couldn't negotiate its own value to transfer, and it couldn't be minted. I tried reaching out to base (the host of onchainsummer), and they ignored me. Coinbase, finally after 13 months, took the wallets down. I believe it was fraudulent to. It was booby trapped on other coding apps. I was told to put it on hold, I never heard anything about it.

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.

6 participants