-
Notifications
You must be signed in to change notification settings - Fork 0
Documents proxy contracts in README #6
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I get a sense you tried to document this for audit, but I'm not sure we have to chatty. The contracts here are rather simple and we need to explain we designed them to be composable and maybe show some examples. E.g. building wstETH/USD
from Api3 feeds.
For the user docs, what I have in mind:
- Document how to run the scripts to deploy the each of the combinators
- Have some common use cases
Btw. I had to look deep into implementation of @api3/contracts to understand that I should populate In other words, for the user. Assume a person who knows that this repo is used to combine Api3 proxies and wants to do it for their dApp. They have no idea about any Api3 internals. It also needs to say that this is provided without a warranty and there is no coverage. |
Yes, I tried giving as much context as needed for auditors to be able to understand the use-cases and design philosophy without much need of going to other repos/docs. I've now simplified a lot the README based on comments.
Where do you think this user-facing docs should go? Same README file or a spearate file under |
This README and the audit/dev comments to a separate one. Similarly, how it's done in https://github.com/api3dao/api3-dao-dashboard?tab=readme-ov-file#development-instructions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should also note somewhere that the user is NOT expected to commit/create-pr with the deployed files and they act only for their convenience.
Deployment artifacts are added to |
Initial documentation for the combinator proxy contracts