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

state | nodebuilder/state: Implement SubmitPayForBlobWithOptions #3346

Open
renaynay opened this issue Apr 29, 2024 · 0 comments · May be fixed by #3349
Open

state | nodebuilder/state: Implement SubmitPayForBlobWithOptions #3346

renaynay opened this issue Apr 29, 2024 · 0 comments · May be fixed by #3349
Assignees
Labels
area:api Related to celestia-node API area:core_and_app Relationship with Core node and Celestia-App area:state Related to fetching state and state execution

Comments

@renaynay
Copy link
Member

Implement an endpoint in the CoreAccessor that uses the functionality of SubmitPayForBlob but contains an PayForBlobOptions struct which it can read from in order to configure some options (for example, the ability to specify an account from which you'd like to submit the blob).

For the initial implementation of this endpoint, the struct can be empty. We will populate it with the first option as part of #3295.

This endpoint + the options struct should be documented clearly and added to the state module's API.

Please do not forget to regenerate mocks + openrpc spec

This PR should not be breaking as it is only an additional endpoint.

The docs for using SubmitPayForBlobWithOptions can be populated in github.com/celestiaorg/docs once multi accounts is finished.

@renaynay renaynay added area:state Related to fetching state and state execution area:core_and_app Relationship with Core node and Celestia-App area:api Related to celestia-node API labels Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:api Related to celestia-node API area:core_and_app Relationship with Core node and Celestia-App area:state Related to fetching state and state execution
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants