Skip to content

Conversation

@Joeysantoro
Copy link

When opening a pull request to submit a new EIP, please use the suggested template: https://github.com/ethereum/EIPs/blob/master/eip-template.md

We have a GitHub bot that automatically merges some PRs. It will merge yours immediately if certain criteria are met:

  • The PR edits only existing draft PRs.
  • The build passes.
  • Your GitHub username or email address is listed in the 'author' header of all affected PRs, inside .
  • If matching on email address, the email address is the one publicly listed on your GitHub profile.

Joeysantoro and others added 3 commits October 26, 2023 14:34
Co-authored-by: Andrew B Coathup <28278242+abcoathup@users.noreply.github.com>
vikramarun pushed a commit that referenced this pull request Mar 12, 2024
* Update erc-7007.md to add opML compatibility (#1)

* 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

* feat: update implementation to include owner address

* feat: update authorship, added Updatable extension, update metadata with `proof_type`

* fix: full term before abbreviation

* feat: Update ERC7007 contracts and add IERC7007Updatable interface

* feat: Add rationale and security considerations

* feat: `update` function to use `verify` in `require` statement

* misc: capitalization

* xhyu patch 1 (#2)

* 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>

* fix: proposal reference and header

* fix: file references

* Update erc-7007.md: Fix wordings (#4)

* Update erc-7007.md: Fix wordings

* update license

* Update description and motivation -> rationale

---------

Co-authored-by: drCathieSo.eth <socathie@users.noreply.github.com>

* Add workflow (#5)

* Add workflow diagram

* Add workflow description

* Apply suggestions from code review

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* Update erc-7007.md

* Fix uppercase keywords

* Update erc-7007.md

---------

Co-authored-by: 李婷婷 Lee Ting Ting <tina1998612@users.noreply.github.com>
Co-authored-by: xhyumiracle <xhyumiracle@gmail.com>
Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants