search

realitycollective / com.realitytoolkit.core

The all-purpose framework for building the next reality
https://realitytoolkit.realitycollective.net/
MIT License
3 stars 0 forks source link

Remove start scene setting #71

Closed FejZa closed 1 year ago

FejZa commented 1 year ago

Reality Collective - Reality Toolkit Pull Request

Overview

We haven't been enforcing the start scene for a while now. This PR removes what's left of it from the toolkit preferences.

SimonDarksideJ commented 1 year ago

Fine to remove, so long as we do have a plan/guidance for scene management, especially since the ServiceManager "should" be in a base scene to initialise BEFORE the project starts properly.

FejZa commented 1 year ago

That's not true. The service manager "should" nothing. It should do whatever the developer wants it to do. If I want it to initialize only when I load a specific scene of the project, then that's when it should do it.

FejZa commented 1 year ago

It's really up to the developer to use the SF in a way, that fits the project. Obviously the developer has to make sure, the SF has had the time to init before trying to get services from it. But that's why we introduced those events recently.

SimonDarksideJ commented 1 year ago

I'd say we should provide guidance on best practices and patterns we have used in projects, especially for new developers who might not know better.