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

feat(be): integrate block explorer Json-RPC server #41

Merged

Conversation

0xbcdev
Copy link
Contributor

@0xbcdev 0xbcdev commented Apr 2, 2024

This PR integrate the Block Explorer Json-RPC server.
Later, once the Block Explorer service deployed, it can start connecting to the Json-RPC service on RollApp nodes for querying data.

New materials added:

@0xbcdev 0xbcdev requested a review from a team as a code owner April 2, 2024 20:07
@hoangdv2429
Copy link
Contributor

Can you fix lint

Copy link
Contributor

@trinitys7 trinitys7 left a comment

Choose a reason for hiding this comment

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

@0xbcdev I got an error while starting node with json-rpc:
Error: listen tcp: address https://localhost:11100: too many colons in address
Could you please check it first before we can merge it?

@trinitys7
Copy link
Contributor

Also, the wasm rpc repo still containing lots of TODO. Please handle them.

@0xbcdev
Copy link
Contributor Author

0xbcdev commented Apr 3, 2024

Screen Shot 2024-04-03 at 21 47 00

@trinitys7 this is the default value, did you changed it wrongly? (0.0.0.0:11100)

@0xbcdev
Copy link
Contributor Author

0xbcdev commented Apr 3, 2024

Also, the wasm rpc repo still containing lots of TODO. Please handle them.

It is not need to handle at this moment. I put there to do on next phase.
This time, we only need the Block Explorer up. Advanced functions to be resolved later.

@omritoptix omritoptix merged commit 51fd3e3 into dymensionxyz:main Apr 3, 2024
24 checks passed
@0xbcdev 0xbcdev deleted the feat/dym-rollapp-explorer-server branch April 4, 2024 08:03
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