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

UI 1264 pm window #251

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

UI 1264 pm window #251

wants to merge 4 commits into from

Conversation

xGuTeK
Copy link
Contributor

@xGuTeK xGuTeK commented Aug 26, 2024

Description

Issue link - #141

This pull request introduces whisper chat windows and enhances the server-side functionality for message targeting by adding support for message blocking. Currently, this blocking feature is implemented only on the server side, with the client-side lacking this functionality. We will need to address the client-side implementation of message blocking in a future update when we work on refining the main chat window.

I based the server-side function changes on the PDB of Ebenezer 1310, but the implementation of the private chat windows was done without using assembly code. Therefore, there may be differences, and it is necessary to verify against the client's assembly code.

💔 Thank you!
Whisper1
Whisper2

@xGuTeK xGuTeK self-assigned this Aug 26, 2024
@stevewgr
Copy link
Member

Awesome! Thanks for the pull-request @xGuTeK 🚀 As also discussed privately, I'll review it possibly on one of the weekends, because currently I'm unfortunately occupied with work.

@xGuTeK xGuTeK added the official_client_code_review Needs verification against the official client code using IDA/Ghidra label Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
official_client_code_review Needs verification against the official client code using IDA/Ghidra
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants