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

docs: add some IS and CL clarifications #105

Merged
merged 2 commits into from
Mar 27, 2024

Conversation

thephez
Copy link
Collaborator

@thephez thephez commented Mar 27, 2024

Update description of RPC instantlock fields, add links to glossary, and add links to more info from glossary entries

Preview build: https://dash-docs--105.org.readthedocs.build/projects/core/en/105/

@thephez thephez merged commit 4a0da78 into dashpay:20.1.0 Mar 27, 2024
1 check passed
@thephez thephez deleted the v20.1-is-cl-info branch March 27, 2024 14:57
@riongull
Copy link

Thanks for updating the docs with these clarifications, @thephez. This will help devs new to Dash.

@coolaj86
Copy link
Contributor

coolaj86 commented Mar 28, 2024

Note: it would also be good to have "Remote Procedure Calls" named "Remote Procedure Calls (RPC)" or just "RPC" so that when trying to search the docs it's easier to find it in the menu via Ctrl+F / Cmd+F.

(I don't think anyone ever says or refers to RPC as "Remote Procedure Call" outside of an academic setting - much like REST, it's kinda lost it's meaning through it's colloquial use)

@thephez
Copy link
Collaborator Author

thephez commented Mar 28, 2024

Note: it would also be good to have "Remote Procedure Calls" named "Remote Procedure Calls (RPC)" or just "RPC" so that when trying to search the docs it's easier to find it in the menu via Ctrl+F / Cmd+F.

(I don't think anyone ever says or refers to RPC as "Remote Procedure Call" outside of an academic setting - much like REST, it's kinda lost it's meaning through it's colloquial use)

Just to clarify, you're referring to the name of this section?

image

@coolaj86
Copy link
Contributor

Yes. That's correct. Though it's strange that when I searched I don't recall it scrolling to the "Address Index RPCs" either.

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.

3 participants