Added support for both LegacyChatService and TextChatService in regards to player targetted binds

Krcnos commented 9 months ago

This pull request adds support for both LegacyChatService and TextChatService in regards to player targetted binds. By default newly created games utilise TextChatService now and having this on doesn't support the .Chatted player event so the bind functionality therefore needs to be updated to dynamically support both.

Declarations:

[x] I declare that this contribution was created in whole or in part by me.
[x] I declare that I have the right to submit this contribution under the terms of this repository's license and declarations.
[x] I understand and agree that this contribution and a record of it are public, maintained permanently, and may be redistributed under the terms of this repository's license.

autonordev commented 8 months ago

Otherwise these changes look good to me. I'm happy to merge if they pass manual tests (by wil) and CI.

Krcnos commented 8 months ago

Installed stylua and committed changes it made, hopefully that passes

autonordev commented 8 months ago

Thanks for the contribution! @Krcnos

evaera / Cmdr

Added support for both LegacyChatService and TextChatService in regards to player targetted binds #322