-
Notifications
You must be signed in to change notification settings - Fork 1
Description
Verify
- I understand that if I ignore these instructions, my issue may be closed without review.
- I have made sure that there are no similar opened issues.
- I have made sure that the SDK version matches the latest release.
Is your feature request related to a problem? Please describe.
We should consider adding a new SDK.Player.Action module to handle everything related to different player actions. For example, see dstmodders/mod-keep-following#8. It would be nice to have SDK.Player.Action.UseItem() at very least and let SDK respect both client and server.
Describe the solution you'd like
Add SDK.Player.Action similar to SDK.Player.Attribute. Respectively, SDK.Remote.Player should be extended. I would also consider adding 4th level submodules support and split SDK.Remote.Player further:
SDK.Remote.Player.Action
SDK.Remote.Player.AttributeThe existing SDK.Player.WalkToPoint() and SDK.RPC.WalkToPoint() should be moved into SDK.Player.Action and SDK.Remote.Player.Action respectively.
Describe alternatives you've considered
We could leave SDK.RPC as is and not extend SDK.Remote.Player. Another consideration is that we could move all player-related remote functionality:
SDK.Remote.Player.Action => SDK.Player.Remote.Action
SDK.Remote.Player.Attribute => SDK.Player.Remote.AttributeAdditional context
Progress
- Add
SDK.Player.Action - Move and rename
SDK.Remote.PlayertoSDK.Player.Remote - Split
SDK.Player.Remoteinto submodules:-
SDK.Player.Remote.Action -
SDK.Player.Remote.Attribute
-
- Implement
SDK.Player.Remote.Actionfunctions:-
Hide() -
UseItem() -
UseEquippedHeadItem() -
UseEquippedBodyItem() -
WalkToPoint()
-
- Implement
SDK.Player.Actionfunctions:-
Hide() -
UseItem() -
UseEquippedHeadItem() -
UseEquippedBodyItem() -
WalkToPoint()
-
- Write tests