Skip to content

Fix deleted message logging in mod alert#2524

Merged
wookie184 merged 2 commits into
mainfrom
fix_alert_message_logging
Apr 12, 2023
Merged

Fix deleted message logging in mod alert#2524
wookie184 merged 2 commits into
mainfrom
fix_alert_message_logging

Conversation

@mbaruh
Copy link
Copy Markdown
Member

@mbaruh mbaruh commented Apr 10, 2023

The alerting only uploaded the related messages, so if they didn't come including the context message, it wasn't uploaded.

@mbaruh mbaruh added t: bug Something isn't working a: backend Related to internal functionality and utilities (error_handler, logging, security, utils and core) p: 1 - high High Priority a: filters Related to message filters: (antimalware, antispam, filtering, token_remover) labels Apr 10, 2023
@vivekashok1221 vivekashok1221 self-requested a review April 11, 2023 07:37
Copy link
Copy Markdown
Contributor

@wookie184 wookie184 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@wookie184 wookie184 enabled auto-merge April 12, 2023 14:21
@wookie184 wookie184 merged commit ae56736 into main Apr 12, 2023
@wookie184 wookie184 deleted the fix_alert_message_logging branch April 12, 2023 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: backend Related to internal functionality and utilities (error_handler, logging, security, utils and core) a: filters Related to message filters: (antimalware, antispam, filtering, token_remover) p: 1 - high High Priority t: bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants