⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@vcarl
Copy link
Member

@vcarl vcarl commented Jan 15, 2026

It was trying to record things to the db, but tbh that's not really necessary here. Useful, but there's a lot of code rn that relies on having a message available, and there isn't one if it was blocked

It was trying to record things to the db, but tbh that's not really necessary here. Useful, but there's a lot of code rn that relies on having a message available, and there isn't one if it was blocked
@vcarl vcarl marked this pull request as ready for review January 15, 2026 21:36
@what-the-diff
Copy link

what-the-diff bot commented Jan 15, 2026

PR Summary

  • Refined reportAutomod Function
    The reportAutomod function, an integral part of our system, has undergone a bit of housekeeping. We've removed a parameter known as messageId from its signature. This means it will now operate more efficiently, without needing to handle or consider this parameter.

  • Removed Redundant Database Interactions
    Part of the update involved simplifying how we handle reports in our database. Previously, whenever a messageId was provided, we would store a related report in the database, complete with error handling and retry logic. We've now eliminated this entire block of code. This is going to make for a quicker, more streamlined interaction with the database.

@vcarl vcarl merged commit fc6c547 into main Jan 15, 2026
6 checks passed
@vcarl vcarl deleted the vc-fix-dupe-threads branch January 15, 2026 21:38
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.

2 participants