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

Minor update to json-rpc.md #4001

Merged
merged 2 commits into from
Aug 15, 2022
Merged

Minor update to json-rpc.md #4001

merged 2 commits into from
Aug 15, 2022

Conversation

patrickkuo
Copy link
Contributor

@patrickkuo patrickkuo commented Aug 15, 2022

Removed sync account example it will be deprecated soon.
removed dups In the following sections we will show how to use Sui's JSON-RPC API.....
use proper headers instead of rpc method name

@github-actions github-actions bot added the Type: Documentation Improvements or additions to documentation label Aug 15, 2022
@patrickkuo
Copy link
Contributor Author

FYI @Romainua

@Clay-Mysten Clay-Mysten self-assigned this Aug 15, 2022
@Clay-Mysten Clay-Mysten removed the request for review from randall-Mysten August 15, 2022 17:17
Copy link
Contributor

@Clay-Mysten Clay-Mysten left a comment

Choose a reason for hiding this comment

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

Thanks, Patrick! This LGTM with some small changes I made in a commit here.

@Clay-Mysten
Copy link
Contributor

FYI @Romainua

@Romainua
Copy link
Contributor

LGTM, will check in doc and check all output and methods

@Clay-Mysten Clay-Mysten merged commit f39b716 into main Aug 15, 2022
@Clay-Mysten Clay-Mysten deleted the pat/update_json_rpc.md branch August 15, 2022 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants