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

spec: add support for providing a single URL for all EIP-712 schemas #66

Merged
merged 2 commits into from
Oct 24, 2024

Conversation

jnicoulaud-ledger
Copy link
Collaborator

  • schemas property can be uri-reference | array[schema | uri-reference]
  • uri-reference can be a <scheme>:// URL, or directly a relative (or absolute) path (./foo or /foo)
  • Note that our implementation will allow https and file URL schemes, or relative paths. This is not enforced by schema/spec, not sure if it should

- `schemas` property can be `uri-reference | array[schema | uri-reference]`
- `uri-reference` can be a `<scheme>://` URL, or directly a relative (or absolute) path (`./foo` or `/foo`)
- Note that our implementation will allow `https` and `file` URL schemes, or relative paths. This is not enforced by schema/spec, not sure if it should
@github-actions github-actions bot added the specifications Changes to ERC-7730 specifications label Oct 21, 2024
@lcastillo-ledger lcastillo-ledger merged commit a206093 into master Oct 24, 2024
5 checks passed
@lcastillo-ledger lcastillo-ledger deleted the eip-712-schema-urls branch October 24, 2024 09:28
jnicoulaud-ledger added a commit that referenced this pull request Oct 25, 2024
EIP-712 `schemas` can be `url | array[schema | url]`
jnicoulaud-ledger added a commit that referenced this pull request Dec 10, 2024
EIP-712 `schemas` can be `url | array[schema | url]`

Co-authored-by: Frédéric Samier <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
specifications Changes to ERC-7730 specifications
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants