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

fix: allow the check for a NetworkObject component on NetworkBehaviours to be disabled #3031

Conversation

NoelStephensUnity
Copy link
Collaborator

@NoelStephensUnity NoelStephensUnity commented Aug 25, 2024

This PR adds an additional Multiplayer->Netcode for GameObjects property to control whether the in-editor NetworkBehaviour check for an associated NetworkObject is performed.
image

MTTB-373

fix: #2966

Changelog

  • Added: "Check for NetworkObject Component" property to the Multiplayer->Netcode for GameObjects project settings. When disabled, this will bypass the in-editor NetworkObject check on NetworkBehaviour components.

Testing and Documentation

  • No tests have been added.
  • Documentation changes will be determined. (wip)

Adding additional configuration parameter, NetworkBehaviour-Check-For-NetworkObject, that provides user the ability to bypass the automatic check for a NetworkObject on a NetworkBehaviour (in editor only).
Adding project settings UI for the added check for networkobject property.
Exiting early if the check for NetworkObject setting is disabled.
@NoelStephensUnity NoelStephensUnity marked this pull request as ready for review August 26, 2024 14:27
@NoelStephensUnity NoelStephensUnity requested a review from a team as a code owner August 26, 2024 14:27
change log entry
Disable and uncheck the Auto-Add NetworkObject Component when Check For NetworkObject Component is disabled.
Enable Auto-Add NetworkObject Component when Check For NetworkObject Component is re-enabled.
User can select if they want to re-check the Auto-Add NetworkObject Component at that point.
@jabbacakes
Copy link
Contributor

I think we maybe need a leetle doc update here, altho screenshots is probably overkill. See PR #1333 on the docs repo.

@NoelStephensUnity NoelStephensUnity merged commit 21cf55e into develop-2.0.0 Aug 31, 2024
24 checks passed
@NoelStephensUnity NoelStephensUnity deleted the fix/allow-check-for-networkobject-to-be-disabled branch August 31, 2024 22:24
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.

2 participants