-
Notifications
You must be signed in to change notification settings - Fork 17
feat: pre-load assets #335
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
base: experimental
Are you sure you want to change the base?
Conversation
Test this pull request on NPM or Yarn
|
|
|
||
| // AssetLoadLoadingState is set by the engine and provides information about | ||
| // the current state of the AssetLoad of an entity. | ||
| message PBAssetLoadLoadingState { |
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 Grow Only Set ?
if not we should have an array here as well, because if not how we gave granularity in all the assets that we asked to pre load /
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 the same type of PointerEventsResult and will be handled by the client in the same way by appending messages
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.
great !
We should add a comment then like the pointerEventResult that tells that is appending values.
No description provided.