-
Notifications
You must be signed in to change notification settings - Fork 75
Conversation
Remove BlittableBool Fix bug where stopping the game threw an error about removing the authoritative player entity.
@@ -45,7 +46,10 @@ protected override void OnUpdate() | |||
} | |||
else | |||
{ | |||
SessionStatus = sessionGroup.GetComponentDataArray<Session.Component>()[0].Status; | |||
Entities.With(sessionGroup).ForEach((ref Session.Component session) => |
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.
is there any chance that we have more than 1 entity matching this query?
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.
if you misconfigure it ><
but you should always have at most one session entity in your deployment.
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.
@jessicafalk I think no?
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.
as mentioned: our snapshot always only contains at most one entity. A user can of course mess around with it and change it, but we assume you only have it at most once
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.
Is it oke by you to leave it like this?
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's not nice, but ok to leave it like this for now. It's not a big issue
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.
generally lgtm
Description
Upgrade the FPS to use the new Entities API's.
Tests
Ran locally in every scene.
Cloud deployment is currently launching
Documentation
Changelog updated.