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 Call Preparation API #758

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lukasrosario
Copy link
Contributor

This proposal defines complementary JSON-RPC methods to EIP-5792's wallet_sendCalls. While wallet_sendCalls is used for an app to submit calls to be signed and submitted in a wallet's interface, the methods in this proposal are for an application to request prepared calls (where "prepared" depends on the wallet implementation) to be signed and submitted by the application itself.

@eip-review-bot
Copy link
Collaborator

eip-review-bot commented Dec 6, 2024

File ERCS/erc-draft-wallet-call-preparation-api.md

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

Copy link

github-actions bot commented Dec 6, 2024

The commit 483c202 (as a parent of 3c41594) contains errors.
Please inspect the Run Summary for details.

@github-actions github-actions bot added the w-ci label Dec 6, 2024
@@ -0,0 +1,200 @@
---
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
---
---
eip: 7836

Assigning next sequential EIP/ERC/RIP number.
Numbers are assigned by editors & associates.

Please also update the filename.

title: Wallet Call Preparation API
description: Adds JSON-RPC methods for requesting prepared EIP-5792 calls.
author: Lukas Rosario (@lukasrosario), Conner Swenberg (@ilikesymmetry), Adam Hodges (@ajhodges), Paaras Bhandari (@paarasbhandari)
discussions-to: TODO
Copy link
Contributor

Choose a reason for hiding this comment

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

Please create a discussions topic in Eth Magicians with a link to this PR: https://ethereum-magicians.org/c/ercs/57

#### Parameters

```typescript!
type PrepareCallsParams = [{

Choose a reason for hiding this comment

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

@lukasrosario should we add a 'version' field to the params for these two methods similar to the one in 5792?

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.

4 participants