Skip to content

Conversation

@lbirkert
Copy link

@lbirkert lbirkert commented Aug 9, 2024

Fix #151

This has several benefits:

  • Allows for a more accurate counting of minutes
  • Removes the need for update queries for every session every minute
  • Allows the bot to count minutes even when down

Copy link

@AdamEXu AdamEXu left a comment

Choose a reason for hiding this comment

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

hurry up and approve this code

Copy link

@AdamEXu AdamEXu left a comment

Choose a reason for hiding this comment

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

@techpixel
Copy link
Member

techpixel commented Aug 14, 2024

hey, I can't approve & merge this without thorough testing. I'll send you the necessary creds. I'm testing it locally on my own machine using - I'll let you know if it works or not.

@techpixel techpixel requested review from AdamEXu and matytyma August 14, 2024 01:40
@techpixel techpixel marked this pull request as draft August 14, 2024 01:40
Copy link

@AdamEXu AdamEXu left a comment

Choose a reason for hiding this comment

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

i already said i approved, why did you request a review?

@lbirkert lbirkert marked this pull request as ready for review August 16, 2024 15:33
@lbirkert lbirkert requested a review from rutmanz August 16, 2024 15:34
Copy link
Contributor

@rutmanz rutmanz left a comment

Choose a reason for hiding this comment

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

I haven't tested but the code lgtm

@lbirkert
Copy link
Author

(had to undo the change, as it would be a bug not to, and pixel already said this would be attacked in a future PR)

@techpixel
Copy link
Member

techpixel commented Aug 19, 2024

Tested and the code lgtm. Will merge when I get home. (Merging will require bot downtime)

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.

Using timestamps to record session time instead of a counter

5 participants