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

Little Writing Changes to Humanoid Page #1006

Merged
merged 10 commits into from
Jan 30, 2025
Merged

Little Writing Changes to Humanoid Page #1006

merged 10 commits into from
Jan 30, 2025

Conversation

Ruanitoo
Copy link
Contributor

Changes

In short, I made some changes to the Humanoid page, changing Function word to Method in methods, etc.
I'm aware that the description of ChangeState() is long, but I believe it's easier for beginners to understand.

I have done this Push before, I am just making some changes.

Checks

By submitting your pull request for review, you agree to the following:

  • [✅] This contribution was created in whole or in part by me, and I have the right to submit it under the terms of this repository's open source licenses.
  • [✅] I understand and agree that this contribution and a record of it are public, maintained indefinitely, and may be redistributed under the terms of this repository's open source licenses.
  • [✅] To the best of my knowledge, all proposed changes are accurate.

Ruanitoo and others added 8 commits January 28, 2025 03:38
Updates the ChangeState method description, in order to let the users know that It only works on the NetworkOwner of the character side (If the NetworkOwner is a player, It must be done using a localscript owned by that player. If it's server, must be done in a server script
Updates the ChangeState method description, in order to let the users know that It only works on the NetworkOwner of the character side (If the NetworkOwner is a player, It must be done using a localscript owned by that player. If it's server, must be done in a server script
Updates the ChangeState method description, in order to let the users know that It only works on the NetworkOwner of the character side (If the NetworkOwner is a player, It must be done using a localscript owned by that player. If it's server, must be done in a server script
I made some writing changes to the Humanoid page.
I know that the ChangeState() method has a long description, but I believe it is easier for beginners to understand.
@Ruanitoo Ruanitoo requested a review from a team as a code owner January 30, 2025 09:42
@github-actions github-actions bot added the engine reference Changes the Engine API Reference documentation label Jan 30, 2025
@github-actions github-actions bot added the changes requested This pull request has changes requested prior to merging label Jan 30, 2025
@IgnisRBX
Copy link
Contributor

Turns out I didn't actually commit your changes from the previous pull request (I just resolved the conversation), so I applied them here instead. This looks good to me now... please post a thumbs up if you agree and I'll merge it. :)

@Ruanitoo
Copy link
Contributor Author

Turns out I didn't actually commit your changes from the previous pull request (I just resolved the conversation), so I applied them here instead. This looks good to me now... please post a thumbs up if you agree and I'll merge it. :)

Seems perfect to me

@IgnisRBX IgnisRBX merged commit 601e2c9 into Roblox:main Jan 30, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes requested This pull request has changes requested prior to merging engine reference Changes the Engine API Reference documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants