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

Create README.md for api docs root folder #60

Merged
merged 2 commits into from
Nov 22, 2023
Merged

Create README.md for api docs root folder #60

merged 2 commits into from
Nov 22, 2023

Conversation

wesfloyd
Copy link
Contributor

Creating a simple README to give context to users on the content and purpose of the API docs.

Why are these changes needed?

The reader needs some context on the purpose and layout of the API docs.

Checks

Not applicable for this single README file

Creating a simple Readme to give context to users on the content and purpose of the API docs.
@wesfloyd wesfloyd requested a review from jianoaix November 20, 2023 15:54
- Churner: a hosted service responsible for maintaining the active set of Operators in the EigenDA network based on their delegated TVL.
- Disperser: the hosted service and primary point of interaction for Rollup users.
- Node: individual EigenDA nodes ran on the network by EigenLayer Operators.
- Retriever: a hosted service that retrieves blobs from EigenDA nodes.
Copy link
Contributor

Choose a reason for hiding this comment

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

The Retriever isn't hosted, it's up to users to operationalize.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Can you share more context on what we mean by "users operationalize"?

E.g. should we say "a service that users can run on their own infrastructure, which exposes a gRPC endpoint for retrieval of blobs from EigenDA nodes"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jianoaix comment addressed - retriever description updated

@wesfloyd
Copy link
Contributor Author

Thanks @jianoaix - should I contact anyone to ask for this pr to be merged?

@jianoaix
Copy link
Contributor

Thanks @jianoaix - should I contact anyone to ask for this pr to be merged?

can you address the comment?

@wesfloyd
Copy link
Contributor Author

Thanks @jianoaix - should I contact anyone to ask for this pr to be merged?

can you address the comment?

Yes - apologies, will fix that now. Sorry - Github n00b mistake ;)

@jianoaix jianoaix merged commit e55fae2 into master Nov 22, 2023
5 checks passed
hopeyen pushed a commit that referenced this pull request Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants