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.

Sign up for GitHub

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

Jump to bottom

gRPC TxStatus endpoint #3753

Closed
cmwaters opened this issue Aug 2, 2024 · 0 comments · Fixed by #3754
Closed

gRPC TxStatus endpoint #3753

cmwaters opened this issue Aug 2, 2024 · 0 comments · Fixed by #3754
Labels
needs:triage

Comments

@cmwaters
Copy link
Contributor

cmwaters commented Aug 2, 2024

We now have a tx status endpoit in celestia-core which returns the real time status of a transaction. The TxClient, for whom this new endpoint is chiefly designed around, only relied on grpc. We need to bridge the gap by creating a new gRPC service that proxies to the underlying RPC server
@cmwaters cmwaters mentioned this issue Aug 2, 2024
Merged
mergify bot pushed a commit that referenced this issue Aug 3, 2024
@cmwaters @mergify
feat: gRPC server for TxStatus endpoint (#3754)
ee9412f 
Closes: #3753

This in some ways ties into
#3421

(cherry picked from commit bb4f712)

# Conflicts:
#	app/app.go
#	app/test/std_sdk_test.go
#	app/test/testnode_test.go
@mergify mergify bot mentioned this issue Aug 3, 2024
Merged
mergify bot pushed a commit that referenced this issue Aug 3, 2024
@cmwaters @mergify
feat: gRPC server for TxStatus endpoint (#3754)
9366dd7 
Closes: #3753

This in some ways ties into
#3421

(cherry picked from commit bb4f712)

# Conflicts:
#	app/app.go
#	app/test/std_sdk_test.go
#	app/test/testnode_test.go
@mergify mergify bot mentioned this issue Aug 3, 2024
Merged
cmwaters added a commit that referenced this issue Aug 5, 2024
@mergify @cmwaters
feat: gRPC server for TxStatus endpoint (backport #3754) (#3755)
548e21a 
Closes: #3753

This in some ways ties into
https://github.com/celestiaorg/celestia-app/issues/3421<hr>This is an
automatic backport of pull request #3754 done by
[Mergify](https://mergify.com).

---------

Co-authored-by: Callum Waters <cmwaters19@gmail.com>
cmwaters added a commit that referenced this issue Aug 5, 2024
@mergify @cmwaters
feat: gRPC server for TxStatus endpoint (backport #3754) (#3756)
9544948 
Closes: #3753

This in some ways ties into
https://github.com/celestiaorg/celestia-app/issues/3421<hr>This is an
automatic backport of pull request #3754 done by
[Mergify](https://mergify.com).

---------

Co-authored-by: Callum Waters <cmwaters19@gmail.com>
ninabarbakadze pushed a commit that referenced this issue Aug 7, 2024
@cmwaters @ninabarbakadze
feat: gRPC server for TxStatus endpoint (#3754)
8b7272e 
Closes: #3753

This in some ways ties into
#3421
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
needs:triage
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: gRPC server for TxStatus endpoint celestiaorg/celestia-app
1 participant
@cmwaters