Skip to content

[Wiki] InvokeServerRpc(MyServerRpc, ...) changed to "MyServerRPC" #424

Merged
LukeStampfli merged 1 commit intomasterfrom
TomasKiniulis-tweakdoc-1
Jan 12, 2021
Merged

[Wiki] InvokeServerRpc(MyServerRpc, ...) changed to "MyServerRPC" #424
LukeStampfli merged 1 commit intomasterfrom
TomasKiniulis-tweakdoc-1

Conversation

@TomasKiniulis
Copy link
Copy Markdown
Contributor

@TomasKiniulis TomasKiniulis commented Dec 31, 2020

In the page: https://mlapi.network/wiki/messaging-system/ Convenience Example section
InvokeServerRpc(MyServerRpc, Random.Range(-50, 50)); "Rpc" throws The name 'MyServerRpc' does not exist in the current context and should be changed to capital letters.

InvokeServerRpc(MyServerRpc, Random.Range(-50, 50)); "Rpc" should be written in capital letters otherwise an exception will be thrown
@unity-cla-assistant
Copy link
Copy Markdown

unity-cla-assistant commented Dec 31, 2020

CLA assistant check
All committers have signed the CLA.

Copy link
Copy Markdown
Contributor

@LukeStampfli LukeStampfli left a comment

Choose a reason for hiding this comment

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

Approved, thanks!

@LukeStampfli LukeStampfli self-assigned this Jan 12, 2021
@LukeStampfli LukeStampfli requested a review from lkrell January 12, 2021 16:03
Copy link
Copy Markdown
Contributor

@lkrell lkrell left a comment

Choose a reason for hiding this comment

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

Looks good!

@LukeStampfli LukeStampfli merged commit 7040c88 into master Jan 12, 2021
@LukeStampfli LukeStampfli deleted the TomasKiniulis-tweakdoc-1 branch January 12, 2021 17:14
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