add celenium post a blob tutorial#1702
Conversation
WalkthroughThe document Changes
Assessment against linked issues
Tip Announcements
Recent review detailsConfiguration used: CodeRabbit UI Files selected for processing (1)
Additional context usedLanguageTool
Additional comments not posted (1)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 0
Outside diff range, codebase verification and nitpick comments (1)
developers/submit-data.md (1)
364-379: Excellent addition! Just a minor nitpick.The new section provides clear and concise instructions for submitting blobs directly from the Celenium explorer interface, enhancing the usability of the document.
Nitpick:
At line 377, replace "the its" with "its" for grammatical correctness.-Once the blob is submitted, you will see the its hash on the screen. +Once the blob is submitted, you will see its hash on the screen.Tools
LanguageTool
[uncategorized] ~372-~372: Consider adding a hyphen.
Context: ... 2. Click on the terminal button in the top right corner of the screen and select the "Su...(TOP_LEFT_CORNER)
[grammar] ~377-~377: A determiner cannot be combined with a possessive pronoun. Did you mean simply “the” or “its”?
Context: ...nce the blob is submitted, you will see the its hash on the screen. You can also use Ce...(A_MY)
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- developers/submit-data.md (2 hunks)
Additional context used
LanguageTool
developers/submit-data.md
[uncategorized] ~372-~372: Consider adding a hyphen.
Context: ... 2. Click on the terminal button in the top right corner of the screen and select the "Su...(TOP_LEFT_CORNER)
[grammar] ~377-~377: A determiner cannot be combined with a possessive pronoun. Did you mean simply “the” or “its”?
Context: ...nce the blob is submitted, you will see the its hash on the screen. You can also use Ce...(A_MY)
Additional comments not posted (1)
developers/submit-data.md (1)
177-177: LGTM!The change is approved as it accurately reflects the expanded functionality for creating and submitting
BlobTxs.
There was a problem hiding this comment.
Actionable comments posted: 0
Outside diff range, codebase verification and nitpick comments (1)
developers/submit-data.md (1)
364-379: Excellent addition of the new section on submitting blobs via Celenium!The new section is well-written and provides clear step-by-step instructions for users to submit blobs directly from the Celenium explorer interface. This is a valuable addition to the document as it introduces a user-friendly method for blob submission.
Just a minor nitpick, feel free to ignore:
Consider adding a hyphen in "top right" to make it "top-right" for consistency with common usage. For example:
- 2. Click on the terminal button in the top right corner of the screen and select the "Submit data blob" option. + 2. Click on the terminal button in the top-right corner of the screen and select the "Submit data blob" option.Overall, great work on enhancing the document with this new section!
Tools
LanguageTool
[uncategorized] ~364-~364: Possible missing comma found.
Context: ...ps://node-rpc-docs.celestia.org/). ### Post a blob directly from Celenium Celenium...(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~372-~372: Consider adding a hyphen.
Context: ... 2. Click on the terminal button in the top right corner of the screen and select the "Su...(TOP_LEFT_CORNER)
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- developers/submit-data.md (2 hunks)
Additional context used
LanguageTool
developers/submit-data.md
[uncategorized] ~364-~364: Possible missing comma found.
Context: ...ps://node-rpc-docs.celestia.org/). ### Post a blob directly from Celenium Celenium...(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~372-~372: Consider adding a hyphen.
Context: ... 2. Click on the terminal button in the top right corner of the screen and select the "Su...(TOP_LEFT_CORNER)
Additional comments not posted (1)
developers/submit-data.md (1)
177-177: LGTM!The change is approved as it accurately reflects the addition of the new blob submission method via Celenium.
| ## API | ||
|
|
||
| Users can currently create and submit `BlobTx`s in four ways. | ||
| Users can currently create and submit `BlobTx`s in six ways. |
There was a problem hiding this comment.
did this PR add 2 ways, or is the old version wrong? thanks!
There was a problem hiding this comment.
did this PR add 2 ways, or is the old version wrong? thanks!
There were five methods listed already, not four.
|
|
||
| Learn more in the [celestia-node API docs](https://node-rpc-docs.celestia.org/). | ||
|
|
||
| ### Post a blob directly from Celenium |
closes #1548
Summary by CodeRabbit