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

Adds in the ability to set a non-networked parent entity for networked NAF entities #440

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

travaglinit
Copy link

@travaglinit travaglinit commented Nov 29, 2023

This is a fork of the changes made in this PR #401 with the addition of bug fixes. This change adds in the ability to set a non-networked parent on networked entities, as described in detail in the PR above.

Addresses issue #400

* Allow networked entities to become childeren of a non-networked parent

* update basic example with commennts

* require # to be in param

* add platform example and update to use syncData.parent field

* quality updates

* quality

* fix css rule

* Address spelling issues

* Add a visible description to the non-networked parent example

* Update instantiation of nonNetworkedParent

---------

Co-authored-by: Akul Gupta <akulgupta2000@gmail.com>
@vincentfretin
Copy link
Member

vincentfretin commented Dec 17, 2023

Thanks @Agupta00 and @travaglinit I'll take a closer look soon. I just released 0.12.0 with some fixes related to the video sharing. I'll integrate this PR in a way or another for 0.13.0.

@travaglinit can you update in your PR the versions for aframe, aframe-extras, environment? see #447 for the versions to use.

@travaglinit
Copy link
Author

Thanks @Agupta00 and @travaglinit I'll take a closer look soon. I just released 0.12.0 with some fixes related to the video sharing. I'll integrate this PR in a way or another for 0.13.0.

@travaglinit can you update in your PR the versions for aframe, aframe-extras, environment? see #447 for the versions to use.

@vincentfretin done! Example in this PR has been updated to use specified versions:

0d839ca

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.

2 participants