Consolidating the name AutomaticallyUnload and UnusedMode in SceneLoadData and SceneUnloadData respectfully. #201
Closed
Katerlad
started this conversation in
Documentation
Replies: 1 comment
-
As we discussed in open development, this isn't going to work. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Hey FGG,
As I am going through the documentation for SceneManagement, I am creating a page about the ability for the server to keep a scene loaded when no clients are observing that scene anymore.
I was hoping we could use this naming convention in the SceneLoadData and SceneUnloadData options instead of currently having two different naming conventions that IMO get confusing.
Suggested
Calling the page "Scene Caching"
SceneLoadData
Current: AutomaticallyUnload - Options: true, false.
Proposed: CacheScenes - Options: true, false
SceneUnloadData
Current: Mode - Options: ServerUnloadMode.UnloadUnused, ServerUnloadMode.KeepUnused.
Proposed: CacheScenes - Options: true, false
Alternative
Alternatively we could call the page "Unused Scenes"
SceneLoadData
Current: AutomaticallyUnload - Options: true, false.
Proposed: UnusedSceneMode - Options: UnusedSceneMode.UnloadUnused, UnloadSceneMode.KeepUnused
SceneUnloadData
Current: Mode - Options: ServerUnloadMode.UnloadUnused, ServerUnloadMode.KeepUnused.
Proposed: UnusedSceneMode - Options: UnusedSceneMode.UnloadUnused, UnloadSceneMode.KeepUnused
Conclusion
In the end I am just hoping we can unify the naming for simplicity no matter what is decided on what the name should be.
Beta Was this translation helpful? Give feedback.
All reactions