-
Notifications
You must be signed in to change notification settings - Fork 811
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
Player Tracking: REST SDK Reference #1570
Player Tracking: REST SDK Reference #1570
Conversation
Build Succeeded 👏 Build Id: d2885726-5357-455c-974e-56cb27946060 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
|
||
#### Alpha: SetPlayerCapacity | ||
|
||
Update the [`GameServer.Status.Players.Capacity`][playerstatus] value with a new capacity. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It might be worth clarifying that this capacity is not enforced as that is not a responsibility of Agones
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a good idea, I'm going to tackle this in #1569 , since it doesn't seem REST API specific.
|
||
With some matchmakers and game matching strategies, it can be important for game servers to mark themselves as `Allocated`. | ||
For those scenarios, this SDK functionality exists. | ||
Apply a Annotation with the prefix "agones.dev/sdk-" to the backing `GameServer` metadata |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apply a Annotation with the prefix "agones.dev/sdk-" to the backing `GameServer` metadata | |
Apply an Annotation with the prefix "agones.dev/sdk-" to the backing `GameServer` metadata |
or the Annotation
not sure what suits better here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. PTAL!
Reference for player tracking API for the REST interface for the SDK. Also regrouped as how we now have it in the Client SDK guide. Work on googleforgames#1033
8197691
to
417f01e
Compare
Build Succeeded 👏 Build Id: 1724e88e-1364-4af1-b355-b54964ef39df The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aLekSer, markmandel The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Build Succeeded 👏 Build Id: 971cb3fb-ba86-4065-8379-391381788701 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
Reference for player tracking API for the REST interface for the SDK. Also regrouped as how we now have it in the Client SDK guide. Work on googleforgames#1033
What type of PR is this?
/kind documentation
What this PR does / Why we need it:
Reference for player tracking API for the REST interface for the SDK.
Also regrouped as how we now have it in the Client SDK guide.
Which issue(s) this PR fixes:
Work on #1033
Special notes for your reviewer: