Skip to content

Small refactor of the interaction system. #1261#1478

Open
rohitkulkarni97 wants to merge 4 commits intoRE-SS3D:developfrom
rohitkulkarni97:Issues/Small-refactor-of-the-interaction-system.-#1261
Open

Small refactor of the interaction system. #1261#1478
rohitkulkarni97 wants to merge 4 commits intoRE-SS3D:developfrom
rohitkulkarni97:Issues/Small-refactor-of-the-interaction-system.-#1261

Conversation

@rohitkulkarni97
Copy link
Contributor

@rohitkulkarni97 rohitkulkarni97 commented Jan 20, 2026

Summary

The Interaction.cs class is removed, and default functions have been added to the IInteraction interface. The IInteraction interface has been divided into IDelayedInteraction and IClientInteractionSource interfaces.

PR checklist

  • The game builds properly without errors.
  • No unrelated changes are present.
  • No "trash" files are committed.
  • Relevant code is documented.
  • Update the related GitBook document, or create a new one if needed.

Pictures/Videos)

Testing

Networking checklist

  • Works from host in host mode.
  • Works from server in server mode.
  • Works on server in client mode.
  • Works and is syncronized across different clients.
  • Is persistent.

Changes

The Interaction class has been removed as it was redundant; instead IInteraction interface has default function implementations which are used.
The IInteraction interface had many unused functions like Update, Cancel, and CreateClients, which were not used by all the classes inheriting from it. Thus, two new interfaces are introduced: IDelayedInteraction for Update and Cancel functions, which are used to handle delayed interactions, and IClientInteractionSource, which has CreateClient function with a default implementation, which is used by different interaction classes to provide ClientInteraction objects.

Related issues/PRs

Closes #1261
Small refactor of the interaction system.

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.

Small refactor of the interaction system.

1 participant