Skip to content

feat: updating API for parity with celestia-node v0.15.0#340

Merged
zvolin merged 11 commits intocelestiaorg:mainfrom
distractedm1nd:txoptions
Aug 22, 2024
Merged

feat: updating API for parity with celestia-node v0.15.0#340
zvolin merged 11 commits intocelestiaorg:mainfrom
distractedm1nd:txoptions

Conversation

@distractedm1nd
Copy link
Contributor

Not ready for merge until node v0.15.0 is released, CI will also fail until then.

  • Adds TxConfig for blob and state modules
  • Removes GasPrice type and gas_prize_serializer
  • Removes SubmitTx endpoint from state module

@zvolin zvolin requested a review from fl0rek August 21, 2024 18:19
@distractedm1nd
Copy link
Contributor Author

We're going to still need blob.Subscribe but that could also be a secondary PR

@zvolin
Copy link
Collaborator

zvolin commented Aug 21, 2024

should be easy enough to add it right now

Copy link
Contributor

@fl0rek fl0rek left a comment

Choose a reason for hiding this comment

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

nice 🙌

Co-authored-by: Mikołaj Florkiewicz <mikolaj@florkiewicz.me>
Signed-off-by: Maciej Zwoliński <mac.zwolinski@gmail.com>
Copy link
Contributor

@fl0rek fl0rek left a comment

Choose a reason for hiding this comment

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

LGTM 🚢

@zvolin zvolin merged commit d9698d2 into celestiaorg:main Aug 22, 2024
@zvolin zvolin mentioned this pull request Aug 22, 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.

4 participants