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

Port chat reliability subsystem from /tg/ #10367

Merged
merged 3 commits into from
Feb 15, 2024

Conversation

Absolucy
Copy link
Contributor

@Absolucy Absolucy commented Dec 21, 2023

About The Pull Request

Ports tgstation/tgstation#79479, tgstation/tgstation#79819, and tgstation/tgstation#79821.

Fixes #9050

Why It's Good For The Game

Should get rid of that annoying bug where random chat messages rarely get "swallowed" and never appear for certain people.

Testing Photographs and Procedure

Screenshots&Videos

2023-12-21 (1703144126) ~ dreamseeker

Changelog

🆑 ZephyrTFA, Absolucy
add: Chat Reliability Layer
code: TGUI chat messages now track their sequence and will be resent if the client notices a discrepenency
/:cl:

@github-actions github-actions bot added the TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0 label Dec 21, 2023
@EvilDragonfiend EvilDragonfiend added Test Merged This PR is currently in rotation Testmerge Passed Seemed fine in testmerge and removed Test Merged This PR is currently in rotation Testmerge Passed Seemed fine in testmerge labels Dec 29, 2023
@EvilDragonfiend
Copy link
Contributor

eh, mistake. it really didn't pass.

image
image
image

@EvilDragonfiend EvilDragonfiend added the Test Merged This PR is currently in rotation label Dec 29, 2023
@itsmeow
Copy link
Member

itsmeow commented Jan 2, 2024

@PowerfulBacon PowerfulBacon removed the Test Merged This PR is currently in rotation label Jan 2, 2024
@Absolucy
Copy link
Contributor Author

Absolucy commented Jan 9, 2024

@itsmeow this already has that lol

@PowerfulBacon
Copy link
Member

Is this good to go?

@PowerfulBacon
Copy link
Member

PowerfulBacon commented Feb 15, 2024

@itsmeow Code owner review

@itsmeow itsmeow added this pull request to the merge queue Feb 15, 2024
Merged via the queue into BeeStation:master with commit 7740bac Feb 15, 2024
8 checks passed
@Absolucy Absolucy deleted the no-more-swiping-my-messages branch February 15, 2024 19:42
DrDuckedGoose pushed a commit to DrDuckedGoose/BeeStation-Hornet that referenced this pull request May 11, 2024
* Port chat reliability subsystem from /tg/

* This crash isn't needed, seems to just be a /tg/ "cleanup" runtime according to git blame
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Improvement Feature Needs Testmerge TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

a chat message sometimes doesn't appear on TGUI chat, but appears on runechat
4 participants