Skip to content

chore: Replace IsSceneObject with InScenePlaced bool#4000

Open
EmandM wants to merge 5 commits into
develop-2.0.0from
chore/replace-issceneobject-with-insceneplaced
Open

chore: Replace IsSceneObject with InScenePlaced bool#4000
EmandM wants to merge 5 commits into
develop-2.0.0from
chore/replace-issceneobject-with-insceneplaced

Conversation

@EmandM
Copy link
Copy Markdown
Member

@EmandM EmandM commented May 25, 2026

Purpose of this PR

Replace the bool? IsSceneObject that was calculated at runtime with a bool InScenePlaced field that's calculated in the asset post processor and a build system callback.

Jira ticket

n/a

Changelog

  • Added: NetworkObject.InScenePlaced boolean for checking if an object is InScenePlaced.
  • Deprecated: NetworkObject.IsSceneObject nullable boolean.

Deprecated API

  • An [Obsolete] attribute was added along with a (RemovedAfter yyyy-mm-dd) entry.
  • An api updater was added.
  • Deprecation of the API is explained in the CHANGELOG.
  • The users can understand why this API was removed and what they should use instead.

Documentation

  • No documentation changes or additions were necessary.
  • Includes documentation for previously-undocumented public API entry points.
  • Includes edits to existing public API documentation.

Testing & QA (How your changes can be verified during release Playtest)

Functional Testing

Manual testing :

  • Manual testing done

Automated tests:

  • Covered by existing automated tests
  • Covered by new automated tests

Does the change require QA team to:

  • Review automated tests?
  • Execute manual tests?
  • Provide feedback about the PR?

If any boxes above are checked the QA team will be automatically added as a PR reviewer.

Backports

n/a

@EmandM EmandM requested a review from a team as a code owner May 25, 2026 22:35
@svc-netcode-sdk svc-netcode-sdk requested a review from a team May 25, 2026 22:36
@EmandM EmandM requested a review from NoelStephensUnity as a code owner May 25, 2026 23:08
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