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 ERC: Wallet Transaction Intents #681

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

ScreamingHawk
Copy link
Contributor

This ERC introduces a mechanism for dapps to provide prerequisites for transactions to smart contract wallets. Wallets can use these prerequisites to make more informed decisions about transaction construction.

@eip-review-bot
Copy link
Collaborator

eip-review-bot commented Oct 21, 2024

File ERCS/erc-7795.md

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

@eip-review-bot eip-review-bot changed the title Add Wallet Transaction Intents draft Add ERC: Wallet Transaction Intents Oct 21, 2024
@github-actions github-actions bot added the w-ci label Oct 21, 2024
ERCS/erc-TBA.md Outdated Show resolved Hide resolved
ERCS/erc-TBA.md Outdated Show resolved Hide resolved
@github-actions github-actions bot removed the w-ci label Oct 23, 2024
Copy link

The commit 13d70b5 (as a parent of 72af9a2) contains errors.
Please inspect the Run Summary for details.

@github-actions github-actions bot added the w-ci label Oct 31, 2024
ERCS/erc-TBA.md Outdated
"type": "ERC20_MIN_BALANCE",
"version": 1,
"payload": {
// See below...

Choose a reason for hiding this comment

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

03f247e2ac8061e11176c8d0526f8a8af3aebffe

ERCS/erc-TBA.md Outdated
"type": "ERC20_MIN_ALLOWANCE",
"version": 1,
"payload": {
"token": "0xA0b86991C6218b36c1d19D4a2e9EB0cE3606eB48",

Choose a reason for hiding this comment

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

03f247e2ac8061e11176c8d0526f8a8af3aebffe

ERCS/erc-TBA.md Outdated
"version": 1,
"payload": {
"token": "0xA0b86991C6218b36c1d19D4a2e9EB0cE3606eB48",
"owner": "0x1234567890abcdef1234567890abcdef12345678",

Choose a reason for hiding this comment

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

03f247e2ac8061e11176c8d0526f8a8af3aebffe

ERCS/erc-TBA.md Outdated

#### ERC721 Ownership

Requires that a given address MUST own a specific [ERC-721](./eip-721.md) token.

Choose a reason for hiding this comment

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

03f247e2ac8061e11176c8d0526f8a8af3aebffe

ERCS/erc-TBA.md Outdated
},
"spender": {
"type": "string",
"pattern": "^0x[a-fA-F0-9]{40}$",

Choose a reason for hiding this comment

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

03f247e2ac8061e11176c8d0526f8a8af3aebffe

ERCS/erc-TBA.md Outdated
"tokenId": {
"type": "string",
"pattern": "^[0-9]+$",
"description": "The ID of the token that must be approved."

Choose a reason for hiding this comment

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

03f247e2ac8061e11176c8d0526f8a8af3aebffe

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.

5 participants