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

fix(cmd/rpc): plaintext parsing of data for blob.Submit and state.SubmitPayForBlob #2340

Merged
merged 1 commit into from
Jun 8, 2023

Conversation

distractedm1nd
Copy link
Member

Closes #2323

@distractedm1nd distractedm1nd added kind:fix Attached to bug-fixing PRs area:api Related to celestia-node API labels Jun 7, 2023
@distractedm1nd distractedm1nd self-assigned this Jun 7, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #2340 (4ee1e5e) into main (a2685d4) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #2340      +/-   ##
==========================================
- Coverage   50.86%   50.83%   -0.04%     
==========================================
  Files         154      154              
  Lines        9739     9743       +4     
==========================================
- Hits         4954     4953       -1     
- Misses       4353     4358       +5     
  Partials      432      432              
Impacted Files Coverage Δ
cmd/celestia/rpc.go 9.82% <0.00%> (-0.12%) ⬇️

... and 1 file with indirect coverage changes

@distractedm1nd distractedm1nd merged commit ae8db72 into main Jun 8, 2023
@distractedm1nd distractedm1nd deleted the submit-cli-plaintext branch June 8, 2023 06:58
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 kind:fix Attached to bug-fixing PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

issue using blob CLI method to submit with data '"gm"'
4 participants