Skip to content
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

Scene Emotes don't work with Local Scene Development #2249

Open
AlejandroAlvarezMelucciDCL opened this issue Oct 1, 2024 · 0 comments · May be fixed by #2355
Open

Scene Emotes don't work with Local Scene Development #2249

AlejandroAlvarezMelucciDCL opened this issue Oct 1, 2024 · 0 comments · May be fixed by #2355
Assignees
Labels
1-high Very important but not critical or game breaking bug Something isn't working sdk

Comments

@AlejandroAlvarezMelucciDCL
Copy link
Collaborator

Scene Emotes not working was planned to be fixed in v3 of support local gltf but this fix is not part of the same systems involved in local gltf support and it's pretty complex so it needs to be tackled in a separated issue.

Emotes triggered by a scene follow an intricate flow using RestrictedActions, a leftover from SDK6.
Some of the related classes are:

This flow is heavily coupled with AssetBundles and needs to be abstracted to support either AssetBundles or local GLBs like it was done in GLTFContainer.
A simpler and faster approach would be to branch the flow depending if we're in Local Scene Development or not, and at a later stage implement a proper abstraction.

@AlejandroAlvarezMelucciDCL AlejandroAlvarezMelucciDCL added the bug Something isn't working label Oct 1, 2024
@github-actions github-actions bot added the new Issues to triage label Oct 1, 2024
@aixaCode aixaCode added 1-high Very important but not critical or game breaking sdk and removed new Issues to triage labels Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1-high Very important but not critical or game breaking bug Something isn't working sdk
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants