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: Common Quote Oracle #500

Merged
merged 31 commits into from
Aug 20, 2024
Merged

Add ERC: Common Quote Oracle #500

merged 31 commits into from
Aug 20, 2024

Conversation

alcueca
Copy link
Contributor

@alcueca alcueca commented Jun 20, 2024

For a while now a bunch of us have been toiling on what is now a minimal standard for oracle value feeds. This all probably started when I wrote this article about using value conversions instead of prices in smart contracts:

In very short, this is a minimal standard with a single function to get the value of an amount of an asset in terms of another asset. There are a number of adapters for current oracles from the community and from Euler.

@alcueca alcueca requested a review from eip-review-bot as a code owner June 20, 2024 10:50
@eip-review-bot
Copy link
Collaborator

eip-review-bot commented Jun 20, 2024

✅ All reviewers have approved.

@github-actions github-actions bot added the w-ci label Jun 20, 2024
@eip-review-bot eip-review-bot changed the title Oracle ERC draft Add ERC: Quote Oracle Standard Jun 20, 2024
@github-actions github-actions bot removed the w-ci label Jun 20, 2024
@eip-review-bot eip-review-bot changed the title Add ERC: Quote Oracle Standard Add ERC: Common Quote Oracle Jun 20, 2024
@alcueca
Copy link
Contributor Author

alcueca commented Jun 20, 2024

Oh great, now it seems to be stuck. Could I get some help here?

@github-actions github-actions bot added the w-ci label Jun 20, 2024
ERCS/erc-7777.md Outdated Show resolved Hide resolved
ERCS/erc-7777.md Outdated Show resolved Hide resolved
ERCS/erc-7777.md Outdated Show resolved Hide resolved
ERCS/erc-7777.md Outdated Show resolved Hide resolved
ERCS/erc-7777.md Outdated Show resolved Hide resolved
@github-actions github-actions bot removed the w-ci label Jun 21, 2024
ERCS/erc-7726.md Outdated Show resolved Hide resolved
ERCS/erc-7726.md Outdated Show resolved Hide resolved
ERCS/erc-7726.md Outdated
Comment on lines 16 to 17
The following standard allows for the implementation of a standard API for data feeds providing the relative value of
assets in the Ethereum and EVM-compatible blockchains.
Copy link
Contributor

Choose a reason for hiding this comment

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

Your abstract does a good job introducing the goals of the proposal, but it lacks a high level (but still technical) overview of the content. A good abstract should have a good amount of detail on how the proposal accomplishes its goals.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I included a bit more technical detail in the abstract. The ERC is very simple, so that limits what I can do.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

If necessary, I can add more content to the abstract from the Euler Whitepaper and my own article introductions, but that feels more like what I would do to an article than a technical abstract.

ERCS/erc-7726.md Outdated Show resolved Hide resolved
ERCS/erc-7726.md Outdated

For BTC, the address will be `0xbBbBBBBbbBBBbbbBbbBbbbbBBbBbbbbBbBbbBBbB`.

For assets without an address, but with an ISO 4217 code, the code will be used (e.g. `address(840)` for USD).
Copy link
Contributor

Choose a reason for hiding this comment

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

ISO standards are not usually free to access, and so we do not permit links to them. Please remove the reference.

Copy link
Contributor

@ruvaag ruvaag Jul 11, 2024

Choose a reason for hiding this comment

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

ISO 4217 in particular is free to use with many open source datasets for access since it's used widely

Copy link
Contributor Author

@alcueca alcueca Jul 11, 2024

Choose a reason for hiding this comment

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

ISO 4217 is explicitly free of charge.

image

The ISO 4217 currency codes have made it widely into the public domain, most notably as part of the chainlink contracts, but also on the wikipedia and other domains.

ERCS/erc-7726.md Outdated Show resolved Hide resolved
@alcueca
Copy link
Contributor Author

alcueca commented Jul 11, 2024

@abcoathup, do you have any idea how to debug that jekyll job that is failing? There is not much between the commit that was working, and the commit where it started not to work.

SamWilsn
SamWilsn previously approved these changes Aug 20, 2024
ERCS/erc-7726.md Outdated Show resolved Hide resolved
@eip-review-bot eip-review-bot enabled auto-merge (squash) August 20, 2024 14:12
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 previously approved these changes Aug 20, 2024
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...

Co-authored-by: Sam Wilson <[email protected]>
auto-merge was automatically disabled August 20, 2024 14:16

Head branch was pushed to by a user without write access

@alcueca alcueca dismissed stale reviews from eip-review-bot and SamWilsn via 55a0015 August 20, 2024 14:16
@eip-review-bot eip-review-bot enabled auto-merge (squash) August 20, 2024 14:21
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 e84ee2b into ethereum:master Aug 20, 2024
10 of 11 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.

8 participants