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

So tired of the chat manager #1342

Merged
merged 3 commits into from
Oct 14, 2021
Merged

So tired of the chat manager #1342

merged 3 commits into from
Oct 14, 2021

Conversation

Cyberboss
Copy link
Member

🆑
Fixed chat channels not mapping when onlining an instance
/:cl:

- SET THE CHANNELS IN CHANGE SETTINGS OMG
- Log ProcessMessage failure when they happen instead of at server shutdown
@Cyberboss Cyberboss added Fix Fixes incorrect functionality Area: Chat With regard to managing chat bots Release This is a release pull request labels Oct 14, 2021
@Cyberboss Cyberboss added this to the v4.15.5 milestone Oct 14, 2021
@codecov
Copy link

codecov bot commented Oct 14, 2021

Codecov Report

Merging #1342 (839c767) into dev (d13c792) will increase coverage by 0.44%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev    #1342      +/-   ##
==========================================
+ Coverage   91.03%   91.47%   +0.44%     
==========================================
  Files         514      514              
  Lines       98859    98866       +7     
  Branches     2680     2680              
==========================================
+ Hits        90000    90442     +442     
+ Misses       8778     8345     -433     
+ Partials       81       79       -2     

@Cyberboss Cyberboss merged commit aa87dc4 into dev Oct 14, 2021
@Cyberboss Cyberboss deleted the SoTiredOfTheChatManager branch October 14, 2021 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Chat With regard to managing chat bots Fix Fixes incorrect functionality Release This is a release pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant