Skip to content

feat: add option to show/hide due date for chores and tasks#185

Merged
isabellaalstrom merged 1 commit intoisabellaalstrom:masterfrom
Bleialf:feature/show-due-date
Jan 28, 2026
Merged

feat: add option to show/hide due date for chores and tasks#185
isabellaalstrom merged 1 commit intoisabellaalstrom:masterfrom
Bleialf:feature/show-due-date

Conversation

@Bleialf
Copy link
Copy Markdown
Contributor

@Bleialf Bleialf commented Jan 27, 2026

This PR adds a new configuration option show_due_date (defaults to true).

Changes:

  • Added show_due_date to the configuration options in README.md.
  • Implemented logic in grocy-chores-card.js to conditionally render the "Due" date line based on this setting.
  • Default behavior remains unchanged (true).

Use Case:
Allows users to declutter their card by hiding due dates if they only want to see the task list without specific timelines.

Copilot AI review requested due to automatic review settings January 27, 2026 17:49
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new configuration option show_due_date to allow users to hide the due date line for chores and tasks, helping declutter the card display when specific timelines are not needed.

Changes:

  • Added show_due_date boolean configuration option (defaults to true for backward compatibility)
  • Modified _shouldRenderDueInDays method to respect the new configuration setting
  • Updated README.md documentation to include the new option

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
grocy-chores-card.js Added configuration property show_due_date and integrated it into the due date rendering logic
README.md Documented the new show_due_date configuration option with its type, default value, and description

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@isabellaalstrom isabellaalstrom merged commit 61577a4 into isabellaalstrom:master Jan 28, 2026
6 of 7 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