Skip to content

fix: tmpo log doesn't always show all entries with --today and --week flags enabled#89

Merged
DylanDevelops merged 2 commits intoDylanDevelops:mainfrom
mabego:fix/log-today-bug
Mar 19, 2026
Merged

fix: tmpo log doesn't always show all entries with --today and --week flags enabled#89
DylanDevelops merged 2 commits intoDylanDevelops:mainfrom
mabego:fix/log-today-bug

Conversation

@mabego
Copy link
Contributor

@mabego mabego commented Mar 18, 2026

Pull Request Checklist

  • I have read and followed the contribution guidelines.
  • My pull request targets the main branch of tmpo.
  • I have tested these changes locally on my machine.

Closes #88

Description

This pull request fixes an issue where log --today does not output all of the day's time entries for the user's local time.

Copy link
Owner

@DylanDevelops DylanDevelops left a comment

Choose a reason for hiding this comment

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

Hi @mabego! Thank you so much for spotting this bug with the --today flag! I appreciate it. Your changes look great and work perfectly on my end.

Doing some more testing, I realized this issue is also affecting the --week flag. I added a comment on fixing that. Can you implement this as well and push it to the branch? When you do, I'll take another look, and it should be good to merge.

Thanks again!

@DylanDevelops DylanDevelops changed the title Fixes log output to use local time with today flag fix: tmpo log doesn't always show all entries with --today and --week flags enabled Mar 19, 2026
Copy link
Owner

@DylanDevelops DylanDevelops left a comment

Choose a reason for hiding this comment

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

This looks great to me! Thank you so much for your contribution! ❤️

@DylanDevelops DylanDevelops merged commit 0378f15 into DylanDevelops:main Mar 19, 2026
3 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.

log --today does not always output all time entries

2 participants