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 EIP: Box type for EIP-712 messages #8594

Merged
merged 21 commits into from
Jun 25, 2024
Merged

Conversation

frangio
Copy link
Contributor

@frangio frangio commented May 23, 2024

No description provided.

@frangio frangio requested a review from eth-bot as a code owner May 23, 2024 22:03
@github-actions github-actions bot added c-new Creates a brand new proposal s-draft This EIP is a Draft t-interface labels May 23, 2024
@eth-bot
Copy link
Collaborator

eth-bot commented May 23, 2024

File EIPS/eip-7713.md

Requires 1 more reviewers from @axic, @lightclient, @SamWilsn

@eth-bot eth-bot added e-consensus Waiting on editor consensus e-review Waiting on editor to review labels May 23, 2024
@github-actions github-actions bot added the w-ci Waiting on CI to pass label May 23, 2024
@github-actions github-actions bot added w-ci Waiting on CI to pass and removed w-ci Waiting on CI to pass labels May 24, 2024
@eth-bot eth-bot changed the title Add EIP: Box types for EIP-712 messages Add EIP: Box type for EIP-712 messages May 24, 2024
@github-actions github-actions bot removed the w-ci Waiting on CI to pass label May 24, 2024
EIPS/eip-7713.md Outdated

Special-purpose interfaces improve usability and security for the user, but rely on standardized message types such as Permits. This EIP concerns the ability to standardize messages that contain within them parameters of arbitrary type.

A recent example is found in [ERC-7683](https://ercs.ethereum.org/ERCS/erc-7683), which defines a struct with the following member:
Copy link
Contributor Author

@frangio frangio May 24, 2024

Choose a reason for hiding this comment

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

This ERC was created post-split. Is there a guideline to link to ERCs now?

Copy link
Contributor

Choose a reason for hiding this comment

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

It's still ./eip-7683.md. The repos get merged during the build.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That doesn't seem to work. Getting this error:

unable to read file eip-7683.md: Io: JsValue(Error: ENOENT: no such file or directory, open 'EIPS/eip-7683.md'

Copy link
Contributor

Choose a reason for hiding this comment

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

Yikes, that's a bug in our stuff. Opened an issue here: #8689

@github-actions github-actions bot added the w-ci Waiting on CI to pass label May 24, 2024
@frangio frangio changed the title Add EIP: Box type for EIP-712 messages Add EIP-7713: Box type for EIP-712 messages May 28, 2024
@eth-bot eth-bot changed the title Add EIP-7713: Box type for EIP-712 messages Add EIP: Box type for EIP-712 messages Jun 17, 2024
@github-actions github-actions bot removed the w-ci Waiting on CI to pass label Jun 17, 2024
This reverts commit dd596a0.
@github-actions github-actions bot added the w-ci Waiting on CI to pass label Jun 17, 2024
Copy link
Contributor

@SamWilsn SamWilsn left a comment

Choose a reason for hiding this comment

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

Going to force merge this because of the eipw bug. It'll keep happening on every pull request until we get that sorted. Sorry!

@SamWilsn SamWilsn merged commit e1ec3e7 into ethereum:master Jun 25, 2024
9 of 10 checks passed
Copy link

The commit 27c129e (as a parent of 3d3005c) contains errors.
Please inspect the Run Summary for details.

Copy link

@Scamreno Scamreno left a comment

Choose a reason for hiding this comment

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

Thx

Scamreno added a commit to Scamreno/EIPs that referenced this pull request Jun 27, 2024
@frangio frangio deleted the eip-box-712 branch June 27, 2024 16:58
Copy link

@Scamreno Scamreno left a comment

Choose a reason for hiding this comment

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

Thx


```js
{
domain: ...,

Choose a reason for hiding this comment

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

primaryType: 'Envelope',
types: {
Envelope: [
{ name: 'account', type: 'address' },

Choose a reason for hiding this comment

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

Giorgi Meskhidze

primaryType: 'Envelope',
types: {
Envelope: [
{ name: 'account', type: 'address' },

Choose a reason for hiding this comment

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

0xA29BEd6D482668b8eA351932A48D128E272E2a94

Copy link

@AndyAkin AndyAkin left a comment

Choose a reason for hiding this comment

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

ad328bdd-d7dc-44ad-8420-ffba6f2e2614

Copy link

@grattitude grattitude left a comment

Choose a reason for hiding this comment

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

``

@AndyAkin
Copy link

@AndyAkin
Copy link

ad328bdd-d7dc-44ad-8420-ffba6f2e2614

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 e-consensus Waiting on editor consensus e-review Waiting on editor to review s-draft This EIP is a Draft t-interface w-ci Waiting on CI to pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.