Skip to content
This repository was archived by the owner on May 21, 2026. It is now read-only.

Player list#16

Open
EstatoDeviato wants to merge 4 commits into
Killing-Fever-Online:masterfrom
EstatoDeviato:player-list
Open

Player list#16
EstatoDeviato wants to merge 4 commits into
Killing-Fever-Online:masterfrom
EstatoDeviato:player-list

Conversation

@EstatoDeviato

@EstatoDeviato EstatoDeviato commented May 28, 2024

Copy link
Copy Markdown
Collaborator
  • new PL packet. example of how should send information to the client:
    ["PL", ['[CHAR_ID1] "CHAR_NAME1" (CHAR_SHOWNAME1)', '[CHAR_ID2] "CHAR_NAME2" (CHAR_SHOWNAME2)']
    Screenshot 2024-07-22 191216

@Crystalwarrior

Copy link
Copy Markdown
Collaborator

Can you show how the player list looks like in the PR description?

@Crystalwarrior

Copy link
Copy Markdown
Collaborator

actually before we merge this, we should consider the functionality of the player list as well and what it should let you do. We can actually merge down a lot of the current functionality into the player list, for example pairing and muting list could be handled from the player list. And for servers that don't support the player list packet we could have it so the list is populated with all the characters to be able to run commands like pairing/muting etc. which is the default behavior

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants