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

Add back the test for the Rigid Body component #320

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Exairnous
Copy link
Contributor

Adds back the test for the Rigid Body component that was removed when the scene grabbables support was removed (#319).

This is mostly for archival purposes as it won't be needed until rigid bodies are reintroduced, which probably won't be until addons support is merged into the Hubs Client (Hubs-Foundation/hubs#6468).

This is needed because the behavior graphs Blender add-on doesn't currently have tests, so it couldn't be transferred with the rest of the scene grabbables stuff that was moved from the main Blender add-on to the behavior graphs Blender add-on.

Scene grabbables support was reverted (Hubs-Foundation/hubs#6515) in the Hubs client due to bugs and is now only present for the Hubs Client's addons branch and its behavior graphs add-on.  This commit removes the scene grabbables and physics support from the main Blender add-on to maintain compatibility with the current Hubs Client.  A corresponding commit to the behavior graphs Blender add-on adds support for scene grabbables/physics back into the behavior graphs Blender add-on so that the functionality isn't lost and can be used there.

Reverts "Merge pull request #285 from MozillaReality/fix-import-error-reporting"

This reverts commit 7eb5656, reversing
changes made to 87074ec.

Reverts "Merge pull request #282 from MozillaReality/scene-grabbables"

This reverts commit 87074ec, reversing
changes made to 3712349.
Adds back the test for the Rigid Body component that was removed when the scene grabbables support was removed (#319).
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.

1 participant