Adds support for lists in the Unreal SDK #4029#4216
Conversation
|
/gcbrun |
|
Build Succeeded 🥳 Build Id: e19ce66e-028a-43bb-817a-9cb5e707a81d 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: |
markmandel
left a comment
There was a problem hiding this comment.
Sweeeeeet!
We'll need some docs here: https://agones.dev/site/docs/guides/client-sdks/unreal/
But make sure the new docs are wrapped in a feature shortcode: https://agones.dev/site/docs/contribute/documentation-editing-contribution/
I am going to call on past unreal contributors, if you wouldn't mind reviewing if possible:
@alexrudd, @KAllan357, @GloryOfNight, @KiaArmani, @DevChagrins - if one of you could take a look at these code changes and make sure they are good?
|
Sure thing! Will work on the docs and get those in. |
|
@markmandel hey, so I see the game engine client SDKs reference Thanks |
|
/gcbrun |
|
Build Failed 😭 Build Id: 019fcec5-caee-42ae-9cb1-32eb34553461 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
Ok, it looks like some of those list methods in other SDKs and they are using Protobuf Fieldmasks and I'm not sure if that exists using regular old HTTP. Just let me know if matching the REST functionality list is ok for Unreal. |
|
/gcbrun |
|
Build Succeeded 🥳 Build Id: af8e6190-0074-4aa7-8b91-0efdf0c61316 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: |
Let me know if I'm understanding correctly. All the other language SDKs (e.g. node, go, etc) use gRPC for their SDK implementations. The engine SDKs use the REST (And some websocket) APIs to implement the Agones integration. The REST API was primarily built for game engines, since gRPC is game engines is... painful 😱 . So if you are asking "should I be using the REST API here" - then the answer is yes 👍🏻 |
|
/gcbrun |
|
Oh, believe me, I know :) I would love to have gRPC support in Unreal, but that would be a herculean effort with all the platforms they support. I will update the Agones Unreal SDK docs to match the REST API ones, since the functionality matches. Thanks! |
|
Build Failed 😭 Build Id: d729a3ee-23a3-4df2-a509-f509b97023cd Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
/gcbrun |
|
Build Failed 😭 Build Id: 5c588bc8-9ea3-4834-9dc1-dd2a458a59e5 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
/gcbrun |
|
LGTM 👍 |
|
Build Succeeded 🥳 Build Id: a413fa44-fce8-4e84-8428-a6588b2728c0 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: |
|
@keith-miller we're about to cut a release. If you need this in for this release, could you please add the short codes to the docs ASAP? This keeps the static version of our site for version 1.50 correctly showing that this isn't implemented.
|
|
Sure thing!
-
Keith
…On Mon, Jul 28, 2025 at 10:19 AM igooch ***@***.***> wrote:
*igooch* left a comment (agones-dev/agones#4216)
<#4216 (comment)>
@keith-miller <https://github.com/keith-miller> we're about to cut a
release. If you need this in for this release, could you please add the
short codes to the docs ASAP? This keeps the static version of our site for
version 1.50 correctly showing that this isn't implemented.
{{% feature expiryVersion="1.51.0" %}}
existing documentation
{{% /feature %}}
{{% feature publishVersion="1.51.0" %}}
new documentation
{{% /feature %}}
—
Reply to this email directly, view it on GitHub
<#4216 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGAHCW7DDDWBFK3FKOJWDT3KZLQNAVCNFSM6AAAAACBCGWRBCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTCMRYGIZDQOJUGU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
/gcbrun |
igooch
left a comment
There was a problem hiding this comment.
LGTM per #4216 (comment)
|
/gcbrun |
|
Build Failed 😭 Build Id: a88a9bf3-ad1f-497f-b11d-4ecdf6b13204 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
/gcbrun |
|
Build Succeeded 🥳 Build Id: 5f59d7c0-7d29-4420-b276-2fd5c8118759 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: |
What type of PR is this?
/kind feature
What this PR does / Why we need it:
Adds support for lists to the Unreal SDK
Which issue(s) this PR fixes:
Closes #4029
Special notes for your reviewer: