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.

Sign up for GitHub

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

Jump to bottom

Add ERC: Cross-Chain Messaging Gateway #673

Open

Amxx wants to merge 45 commits into ethereum:master from Amxx:new-erc/crosschain-gateway-interface

+200 −0

Contributor

Amxx commented Oct 14, 2024 •

edited

Loading

TODO:

get an ERC number
start discussion thread on eth magicians


          adding draft

9cc03cc

Amxx requested a review from eip-review-bot as a code owner

October 14, 2024 13:05

github-actions bot added c-new s-draft t-erc labels

Collaborator

eip-review-bot commented Oct 14, 2024 •

edited

Loading

File `ERCS/erc-7786.md`

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

eip-review-bot added e-consensus e-review labels

eip-review-bot changed the title ~~Cross-Chain Messaging Gateway~~ Add ERC: Cross-Chain Messaging Gateway

github-actions bot added the w-ci label

Amxx commented

View reviewed changes

ERCS/erc-yyyy.md Outdated Show resolved Hide resolved


          Add link to discussion thread

b932584

Amxx commented

View reviewed changes

ERCS/erc-yyyy.md Outdated Show resolved Hide resolved

Amxx commented

View reviewed changes

ERCS/erc-yyyy.md Outdated Show resolved Hide resolved

Amxx commented

View reviewed changes

ERCS/erc-yyyy.md Outdated Show resolved Hide resolved

Amxx commented

View reviewed changes

ERCS/erc-yyyy.md Outdated Show resolved Hide resolved

Amxx added 2 commits

October 14, 2024 16:19


          last version

ed21209


          Merge remote-tracking branch 'amxx/new-erc/crosschain-gateway-interfa…

e74c9e5

…ce' into new-erc/crosschain-gateway-interface

Amxx mentioned this pull request

ERC-7786 interfaces and receiver OpenZeppelin/openzeppelin-contracts#5255

Closed

3 tasks

abcoathup reviewed

View reviewed changes

ERCS/erc-yyyy.md Outdated Show resolved Hide resolved

abcoathup reviewed

View reviewed changes

ERCS/erc-yyyy.md Outdated Show resolved Hide resolved

Amxx and others added 2 commits

October 16, 2024 09:17


          Update ERCS/erc-yyyy.md

27c6cdb

Co-authored-by: Andrew B Coathup <[email protected]>


          Update ERCS/erc-yyyy.md

91841f1

Co-authored-by: Andrew B Coathup <[email protected]>

github-actions bot removed the w-ci label

Amxx added 2 commits

October 16, 2024 09:19


          rename ERC file and interfaces

d6094bc


          add erc link

2f2f25b

github-actions bot added the w-ci label

Contributor Author

Amxx commented Oct 17, 2024

@abcoathup, We believe the document listed the "Reference" section is really valuable to the reader. Is there any "ERC compliant" way to link to it ?


          Update erc-7786.md

97c02ee

github-actions bot removed the w-ci label

github-actions bot removed the w-ci label


          add diagrams

cfc0b65

github-actions bot added the w-ci label

Amxx commented

View reviewed changes

ERCS/erc-7786.md Outdated Show resolved Hide resolved

Amxx and others added 6 commits

November 5, 2024 16:33


          Apply suggestions from code review

0b3c063


          add _processMessage

a9f313c


          use images

5cd3592


          fix images

3d3c0c7


          Merge branch 'new-erc/crosschain-gateway-interface' into new-erc/cros…

07934da

…schain-gateway-interface


          Merge pull request #2 from frangio/new-erc/crosschain-gateway-interface

35d7ebb

Add diagrams

Amxx commented

View reviewed changes

ERCS/erc-7786.md Outdated Show resolved Hide resolved


          Removing link

67cc254

github-actions bot removed the w-ci label


          Update erc-7786.md

b84079a

github-actions bot added the w-ci label

Contributor

frangio commented Nov 16, 2024

We're looking into simplifying this proposal by removing "Passive Mode".

Amxx commented

View reviewed changes

ERCS/erc-7786.md Outdated Show resolved Hide resolved

Amxx and others added 9 commits

November 20, 2024 16:54


          Update ERCS/erc-7786.md

f297a37


          Remove passive mode

5f72c7d


          Update ERCS/erc-7786.md

4ece363

Co-authored-by: Hadrien Croubois <[email protected]>


          Merge pull request #3 from ernestognw/new-erc/crosschain-gateway-inte…

…rface

Remove passive mode


          Update erc-7786.md


          remove passive mode

2ba7cb1


          remove old paragraph

0d14e9f


          merge


          Merge pull request #4 from frangio/new-erc/crosschain-gateway-interface

63b0084

Remove passive mode follow-up

Amxx commented

View reviewed changes

ERCS/erc-7786.md Outdated Show resolved Hide resolved

github-actions bot commented Nov 28, 2024

The commit 63b0084 (as a parent of 8101e83) contains errors.
Please inspect the Run Summary for details.


          Update ERCS/erc-7786.md

f7976e4

Co-authored-by: Francisco Giordano <[email protected]>

github-actions bot removed the w-ci label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Asfand78690 Asfand78690 approved these changes

cjcobb23 cjcobb23 left review comments

ernestognw ernestognw left review comments

abcoathup abcoathup left review comments

gusjavaz gusjavaz left review comments

frangio frangio left review comments

norswap norswap left review comments

eip-review-bot Awaiting requested review from eip-review-bot eip-review-bot is a code owner

At least 0 approving reviews are required to merge this pull request.

Labels

c-new e-consensus e-review s-draft t-erc