Skip to content

Conversation

@Anujsinghdevx
Copy link
Contributor

Description

  • Fixed overdue task calculation in ReportsView so that tasks with past due dates are correctly counted as overdue.
  • Updated snapshots to reflect the corrected overdue counts in daily, weekly, and monthly reports.
  • Ensures compatibility with parseTaskwarriorDate and isOverdue utilities for proper date parsing and comparison.

Checklist

  • Ran npx prettier --write . (for formatting)
  • Ran gofmt -w . (for Go backend)
  • Ran npm test (for JS/TS testing)
  • Added/updated unit tests, if applicable
  • Verified all tests pass
  • Updated documentation, if needed

Additional Notes

  • Snapshots for ReportsView updated to match the corrected overdue logic.
  • This change ensures that all reports display accurate counts for completed, ongoing, and overdue tasks.

@github-actions
Copy link

github-actions bot commented Jan 1, 2026

Thank you for opening this PR!

Before a maintainer takes a look, it would be really helpful if you could walk through your changes using GitHub's review tools.

Please take a moment to:

  • Check the "Files changed" tab
  • Leave comments on any lines for functions, comments, etc. that are important, non-obvious, or may need attention
  • Clarify decisions you made or areas you might be unsure about and/or any future updates being considered.
  • Finally, submit all the comments!

More information on how to conduct a self review:
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/reviewing-proposed-changes-in-a-pull-request

This helps make the review process smoother and gives us a clearer understanding of your thought process.

Once you've added your self-review, we'll continue from our side. Thank you!

@Anujsinghdevx Anujsinghdevx marked this pull request as draft January 1, 2026 06:55
@Anujsinghdevx Anujsinghdevx marked this pull request as ready for review January 1, 2026 06:58
@github-actions
Copy link

github-actions bot commented Jan 1, 2026

Thank you for opening this PR!

Before a maintainer takes a look, it would be really helpful if you could walk through your changes using GitHub's review tools.

Please take a moment to:

  • Check the "Files changed" tab
  • Leave comments on any lines for functions, comments, etc. that are important, non-obvious, or may need attention
  • Clarify decisions you made or areas you might be unsure about and/or any future updates being considered.
  • Finally, submit all the comments!

More information on how to conduct a self review:
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/reviewing-proposed-changes-in-a-pull-request

This helps make the review process smoother and gives us a clearer understanding of your thought process.

Once you've added your self-review, we'll continue from our side. Thank you!

@Anujsinghdevx Anujsinghdevx marked this pull request as draft January 1, 2026 06:59
@Anujsinghdevx Anujsinghdevx marked this pull request as ready for review January 1, 2026 07:28
@github-actions
Copy link

github-actions bot commented Jan 1, 2026

Thank you for opening this PR!

Before a maintainer takes a look, it would be really helpful if you could walk through your changes using GitHub's review tools.

Please take a moment to:

  • Check the "Files changed" tab
  • Leave comments on any lines for functions, comments, etc. that are important, non-obvious, or may need attention
  • Clarify decisions you made or areas you might be unsure about and/or any future updates being considered.
  • Finally, submit all the comments!

More information on how to conduct a self review:
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/reviewing-proposed-changes-in-a-pull-request

This helps make the review process smoother and gives us a clearer understanding of your thought process.

Once you've added your self-review, we'll continue from our side. Thank you!

@its-me-abhishek
Copy link
Collaborator

This got fixed by #365 .Thanks for the PR!

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