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

Economy balance updates are not logged #6

Closed
ShootingKing-AM opened this issue Nov 11, 2021 · 1 comment
Closed

Economy balance updates are not logged #6

ShootingKing-AM opened this issue Nov 11, 2021 · 1 comment
Assignees

Comments

@ShootingKing-AM
Copy link
Contributor

It seems the conins disbursed would be available in mod_logs but after server config, and some students studying and earning coins, its still not availbale in mod_logs and rather its empty.

Related images are below, yes Students studied AFTER the server was conf'ed with channels, etc.

config-1
config
top
log
perms

@Intery
Copy link
Contributor

Intery commented Nov 12, 2021

The mod_log is a ticket log channel intended as an enhanced audit log for moderation actions, not as an economy tracker. event_log would be more suitable for that, and most of the manual events which modify the user's balance are logged there.

However, I chose not to explicitly log all coins gained at this point, since there are potentially a lot of very spammy economy modifications. For example, each TODO task completed may give coins, and the accountability rewards also yield coins. The intention was to add a complete log for per-user economy data, accessible via command in future.

Further, the time tracking system is currently poll-based, per 5 seconds, and rewards fractional amounts accordingly, which is infeasible to log. However, we are in the process of migrating to a session based study tracker which will simplify this considerably, and we may optionally add the completed sessions to the event log (at least for smaller guilds).

@Intery Intery changed the title Mod-logs is empty Economy balance updates are not logged Nov 12, 2021
@Intery Intery linked a pull request Nov 12, 2021 that will close this issue
@Intery Intery self-assigned this Nov 12, 2021
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 a pull request may close this issue.

2 participants