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

Cleanup runechat callbacks upon deletion #2236

Merged
merged 1 commit into from
Jun 16, 2024

Conversation

Absolucy
Copy link
Member

@Absolucy Absolucy commented Jun 12, 2024

About The Pull Request

image

Should get rid of one of the most common runtime errors that occurs.

Changelog

🆑
fix: Fixed a common runtime error relating to runechat messages and logged out clients.
/:cl:

@DexeeXI
Copy link
Collaborator

DexeeXI commented Jun 12, 2024

Thank you! Less runtimes!

@Absolucy
Copy link
Member Author

this should also reduce memory usage too :)

@DexeeXI
Copy link
Collaborator

DexeeXI commented Jun 12, 2024

Sounds even better.

@Absolucy Absolucy force-pushed the runechat-runtime-cleanup branch from df0cd94 to 2bdb131 Compare June 12, 2024 22:45
@Absolucy Absolucy force-pushed the runechat-runtime-cleanup branch from 2bdb131 to d18f0a2 Compare June 12, 2024 23:23
@Absolucy
Copy link
Member Author

haven't seen any chatmessage runtimes since this was TM'd

@dwasint dwasint merged commit 0bab61a into Monkestation:master Jun 16, 2024
20 checks passed
@Absolucy Absolucy deleted the runechat-runtime-cleanup branch June 16, 2024 21:00
github-actions bot added a commit that referenced this pull request Jun 16, 2024
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.

3 participants