celestiaorg / go-cnc

celestia-node REST client in Go.
Apache License 2.0
15 stars 8 forks source link

chore!: renaming PayForData to PayForBlob #42

Closed distractedm1nd closed 1 year ago

distractedm1nd commented 1 year ago

Overview

celestia-app renames PayForData to PayForBlob in https://github.com/celestiaorg/celestia-app/pull/978 node will rename it with https://github.com/celestiaorg/celestia-node/pull/1582

This PR changes the endpoint called at celestia-node, variable names, documentation, and breaks the client's API. Draft for now because we haven't merged it in node.

TODO:

Checklist

codecov-commenter commented 1 year ago

Codecov Report

Merging #42 (c2f0076) into main (1967ead) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #42   +/-   ##
=======================================
  Coverage   54.54%   54.54%           
=======================================
  Files           2        2           
  Lines          88       88           
=======================================
  Hits           48       48           
  Misses         34       34           
  Partials        6        6           
Impacted Files Coverage Δ
client.go 51.80% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

tzdybal commented 1 year ago

We should release this as v0.3.0.