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

Chat handling updated + language friendly GSMG messages #147

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xXFreeFunXx
Copy link

Chat: Checking the player name with lpi
Mapping is session based
Mapping update on every join
GSMG: Custom messages added to config.json
Player name is taken from GSMG messages

@xXFreeFunXx
Copy link
Author

Now compatible with 7 Days to Die 1.0!

@LakeYS
Copy link
Owner

LakeYS commented Jul 31, 2024

This looks great! Will be testing ASAP.

@LakeYS
Copy link
Owner

LakeYS commented Aug 7, 2024

Tested with it. Join/leave messages worked. I'm not able to get chat messages through due to the following error:

/home/lake/Dev/Discord/Dishorde/index.js:203
      content.name = playerIdNameMap[content.entityId] || "Unknown";
                     ^

ReferenceError: playerIdNameMap is not defined
    at handleMsgFromGame (/home/lake/Dev/Discord/Dishorde/index.js:203:22)
    at Telnet.<anonymous> (/home/lake/Dev/Discord/Dishorde/index.js:781:7)
    at Telnet.emit (node:events:520:28)
    at Socket.<anonymous> (/home/lake/Dev/Discord/Dishorde/node_modules/telnet-client/lib/index.js:84:23)
    at Socket.emit (node:events:520:28)
    at addChunk (node:internal/streams/readable:559:12)
    at readableAddChunkPushByteMode (node:internal/streams/readable:510:3)
    at Readable.push (node:internal/streams/readable:390:5)
    at TCP.onStreamRead (node:internal/stream_base_commons:191:23)

I did see the "Received empty response for player list" message go off before this happened.

Also, within the code: could you or someone translate the comments into English to keep things consistent?

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