Skip to content

fix(pubsub): log moderation_actions with moderator username and id#4592

Merged
sogehige merged 1 commit into
masterfrom
4590
Feb 7, 2021
Merged

fix(pubsub): log moderation_actions with moderator username and id#4592
sogehige merged 1 commit into
masterfrom
4590

Conversation

@sogehige
Copy link
Copy Markdown
Collaborator

@sogehige sogehige commented Feb 7, 2021

Also removed handling by TMI which doesn't provide such informations

Fixes #4590

CHECKLIST

Also removed handling by TMI which doesn't provide such informations

Fixes #4590
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 7, 2021

Codecov Report

Merging #4592 (8ee24de) into master (1fd0388) will decrease coverage by 0.38%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4592      +/-   ##
==========================================
- Coverage   58.02%   57.64%   -0.39%     
==========================================
  Files         268      268              
  Lines       15726    15746      +20     
  Branches     3018     3028      +10     
==========================================
- Hits         9125     9076      -49     
- Misses       6601     6670      +69     
Flag Coverage Δ
mysql 57.53% <0.00%> (-0.02%) ⬇️
postgres 57.43% <0.00%> (-0.45%) ⬇️
sqlite ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/bot/pubsub.ts 19.83% <0.00%> (-4.41%) ⬇️
src/bot/tmi.ts 37.21% <ø> (+0.16%) ⬆️
src/bot/helpers/getMigrationType.ts 80.00% <0.00%> (-20.00%) ⬇️
src/bot/systems/levels.ts 51.19% <0.00%> (-11.12%) ⬇️
src/bot/systems/points.ts 65.55% <0.00%> (-7.70%) ⬇️
src/bot/main.ts 75.00% <0.00%> (+0.92%) ⬆️
src/bot/systems/bets.ts 67.93% <0.00%> (+1.63%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1fd0388...8ee24de. Read the comment docs.

@sogehige sogehige merged commit b81d804 into master Feb 7, 2021
@sogehige sogehige deleted the 4590 branch February 7, 2021 16:32
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.

ignore pubsub followersoff

1 participant