-
Notifications
You must be signed in to change notification settings - Fork 11.3k
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
Refine use of certain terms to better match expectations #2672
Comments
Thanks so much, Emma! Let me take first run at fixing this. |
Hi @emmazzz , I am attaching a text file containing all of the references to all forms of "account" in docs.sui.io. Please take a look and let me know if you have guidance for which references should remain. If not, I will use my own judgement and get plenty of review. Thanks! |
I am making this issue more general to also take on related work. In addition to:
|
And "common" for "simple" too |
Reopening since we did not address the original request in #1, Use "address" instead of "account" in docs issue reported here |
Asking @randall-Mysten to help with the outlier in #1 above. |
Hi Emma, I have finally gotten around to tackling the original account > address swap you requested in: Please note, this change should get careful review so I don't unintentionally change the meaning of our docs. I've added numerous folks to the PR to help. Finally, please note I left many seemingly relevant references to accounts in many pages, most notably: This is required because we have output like this in our code: So some concept of account exists. :-) That said, I reduced many references of "account address" to "address" per your recommendation. Please take a look. Thanks! |
Since Sui doesn't have a concept of "account" natively, it might make sense to consistently use "address" in docs and possibly other places to avoid confusion.
The text was updated successfully, but these errors were encountered: