Skip to content

Conversation

@D3luxee
Copy link
Contributor

@D3luxee D3luxee commented Jun 10, 2025

This adds documentation that explicitly explains that only allocated game servers are protected from termination and that game servers can be terminated at any time in all other states.

Copy link
Contributor

@hloeffler hloeffler left a comment

Choose a reason for hiding this comment

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

While the changes in this PR don't seem to cause any issues, I think there's a larger point of confusion in the documentation that's worth addressing.

The page at https://docs.gamefabric.com/multiplayer-servers/getting-started/using-the-agones-sdk has the heading "Using the Agones SDK". This phrasing makes it sound like using the Agones SDK is optional. However, my understanding is that Agones is a requirement for GameFabric's multiplayer servers.

This could lead to confusion for new users. It might be helpful to update the documentation to make it more explicit that Agones is a necessary component, not just an optional tool."

@D3luxee
Copy link
Contributor Author

D3luxee commented Jun 10, 2025

While the changes in this PR don't seem to cause any issues, I think there's a larger point of confusion in the documentation that's worth addressing.

The page at https://docs.gamefabric.com/multiplayer-servers/getting-started/using-the-agones-sdk has the heading "Using the Agones SDK". This phrasing makes it sound like using the Agones SDK is optional. However, my understanding is that Agones is a requirement for GameFabric's multiplayer servers.

This could lead to confusion for new users. It might be helpful to update the documentation to make it more explicit that Agones is a necessary component, not just an optional tool."

After some back and forth in slack this will be addressed in #103

@D3luxee D3luxee merged commit 39478a5 into main Jun 10, 2025
4 checks passed
@D3luxee D3luxee deleted the shutdown_protection branch June 10, 2025 14:46
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