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

Feature/OcMobProperImplementation #311

Merged
merged 2 commits into from
Mar 25, 2024

Conversation

JucanAndreiDaniel
Copy link
Contributor

Make CreateFire load the vobtree

To test

  1. Turn on particles and check if fire is properly parented (both light and particles)

Checklist

Testing

  • Merged main into this branch and tested with the latest features
  • Tested with PCVR
  • Tested with Pico / Quest

@JucanAndreiDaniel JucanAndreiDaniel added feature New feature quick Just a small improvement, but still worth the time. ;-) labels Feb 14, 2024
@JucanAndreiDaniel JucanAndreiDaniel self-assigned this Feb 14, 2024
@JaXt0r JaXt0r self-requested a review March 21, 2024 22:05
Copy link
Contributor

@JaXt0r JaXt0r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fire is inside the stand flares. Looks good.

@JucanAndreiDaniel JucanAndreiDaniel force-pushed the feature/ocmobfire-proper-implementation branch from 083cfe3 to 2bb9e3f Compare March 23, 2024 23:27
Copy link
Contributor

@JaXt0r JaXt0r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works. But! You have Particles disabled in FeatureFlags. Something you want to add to Production ready or want to have it disabled for releases for now?

Hint: If you want to enable it by default, please also check if the other Particles (e.g. leaves in forest) are working as well.

image

@JucanAndreiDaniel JucanAndreiDaniel merged commit 9eab021 into main Mar 25, 2024
@JucanAndreiDaniel JucanAndreiDaniel deleted the feature/ocmobfire-proper-implementation branch March 26, 2024 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature quick Just a small improvement, but still worth the time. ;-)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants