Skip to content

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

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

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

Conversation

@TomasKiniulis

@TomasKiniulis TomasKiniulis commented Dec 31, 2020

Copy link
Copy Markdown
Contributor

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

unity-cla-assistant commented Dec 31, 2020

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@LukeStampfli LukeStampfli left a comment

Copy link
Copy Markdown
Contributor

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

@lkrell lkrell left a comment

Copy link
Copy Markdown
Contributor

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