Skip to content

fix: stats error handling, logging bug in util, and typos#143

Merged
mpinheir merged 1 commit intoOsProgramadores:masterfrom
alexandrevarga:fix-stats-error-handling
Jan 13, 2026
Merged

fix: stats error handling, logging bug in util, and typos#143
mpinheir merged 1 commit intoOsProgramadores:masterfrom
alexandrevarga:fix-stats-error-handling

Conversation

@alexandrevarga
Copy link
Copy Markdown
Contributor

Improved Error Handling in src/stats.go

Implemented the existing FIXME in the saveStats function.
The function now returns an explicit error when the stats writer is nil, instead of failing silently.

Added the stats_error_nil_writer translation key to both en-us-all.toml and pt-br-all.toml
to ensure localization consistency.

Fixed Logging Bug in src/util.go
Corrected the parameter order in the deleteMessage function's log message.
Previously, message_id and chat_id were swapped in the output.

Typos and Documentation Fixes:
src/main.go: Fixed a typo in the bot's startup log message ("Starting op-bo" to "Starting op-bot").
src/bot.go: Adjusted an inconsistent comment where the group rules URL variable was being described with a generic label.

@mpinheir mpinheir merged commit 01ef16e into OsProgramadores:master Jan 13, 2026
2 checks passed
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