Skip to content

Conversation

@plenarius
Copy link
Collaborator

@plenarius plenarius commented Oct 27, 2020

As requested in #746

I'm not really sure how to best create a deprecation shim so for now it's just a bunch of notices

Note: I also added/(fixed?) targeted messages now being able to be sent on party and dm channel.

@plenarius plenarius marked this pull request as draft October 30, 2020 17:46
@plenarius plenarius changed the title Events: Add Chat [WIP] Events: Add Chat Oct 30, 2020
Move SendMessage into Creature
Move Get|SetChatHearingDistance into Player
Add shims in nwnx_chat for all functionality
Warn of deprecation on Chat constructor and in all functions
Add chat hearing distance test unit to nwnx_player_t
@plenarius plenarius changed the title [WIP] Events: Add Chat Events: Add Chat Nov 2, 2020
@plenarius plenarius marked this pull request as ready for review November 2, 2020 23:18
@plenarius
Copy link
Collaborator Author

plenarius commented Nov 3, 2020

Closes #791 with latest update

@plenarius plenarius marked this pull request as draft November 4, 2020 20:17
@plenarius
Copy link
Collaborator Author

Tabling this as I reflect on some hurdles with module chat script and core nwn chat functionality. This may just have to be a breaking change but I'd like to come up with a document for migration if that's the case.

@mtijanic mtijanic changed the title Events: Add Chat [WIP] Events: Add Chat Aug 21, 2021
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.

nwnx_chat - Enable NWNX_Chat_SendMessage() to do per target message sending on channel NWNX_CHAT_CHANNEL_PLAYER_DM

1 participant