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: Auxiliary Funds Capability #372

Merged

Conversation

lukasrosario
Copy link
Contributor

Defines an EIP-5792 compliant capability that allows wallets to indicate to apps that they have access to funds beyond those that can be accounted for by looking up balances onchain given the wallet's address.

https://ethereum-magicians.org/t/add-erc-auxiliary-funds-capability/19599

@eip-review-bot
Copy link
Collaborator

eip-review-bot commented Apr 9, 2024

✅ All reviewers have approved.

Copy link

The commit 644d832 (as a parent of f9adc05) contains errors.
Please inspect the Run Summary for details.

@github-actions github-actions bot removed the w-ci label Apr 10, 2024
ERCS/erc-7682.md Outdated Show resolved Hide resolved
Co-authored-by: Andrew B Coathup <28278242+abcoathup@users.noreply.github.com>
---
eip: 7682
title: Auxiliary Funds Capability
description: A way for wallets to indicate to apps that they have access to additional funds
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
description: A way for wallets to indicate to apps that they have access to additional funds
description: An ERC-5792 capability allowing wallets to indicate that they have access to additional funds.

Your description was fine before, but I feel like explicitly mentioning 5792 in the description might be useful.


## Abstract

An [EIP-5792](./eip-5792.md) compliant capability that allows wallets to indicate to apps that they have access to funds beyond those that can be accounted for by looking up balances onchain given the wallet's address.
Copy link
Collaborator

Choose a reason for hiding this comment

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

It'd be nice to have a bit more technical detail in the abstract. How do wallets indicate this? Where could these funds come from?

Copy link
Collaborator

@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.

Looks good enough for a draft. Take care of the other comments at your leisure.

@eip-review-bot eip-review-bot enabled auto-merge (squash) June 7, 2024 14:51
Copy link
Collaborator

@eip-review-bot eip-review-bot left a comment

Choose a reason for hiding this comment

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

All Reviewers Have Approved; Performing Automatic Merge...

Copy link
Collaborator

@eip-review-bot eip-review-bot left a comment

Choose a reason for hiding this comment

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

All Reviewers Have Approved; Performing Automatic Merge...

@eip-review-bot eip-review-bot merged commit 90edb42 into ethereum:master Jun 7, 2024
10 of 12 checks passed
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.

None yet

4 participants