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

fix: group by [date] now puts Invalid [date] date as first heading #2595

Merged
merged 3 commits into from Jan 17, 2024

Conversation

claremacrae
Copy link
Collaborator

Description

Make invalid dates more prominent in group by due and friends.

Motivation and Context

How has this been tested?

Fixes #2591.

Screenshots (if appropriate)

image

Types of changes

Changes visible to users:

  • Bug fix (prefix: fix - non-breaking change which fixes an issue)
  • Documentation (prefix: docs - improvements to any documentation content for users)
  • Sample vault (prefix: vault - improvements to the Tasks-Demo sample vault)

Internal changes:

  • Tests (prefix: test - additions and improvements to unit tests and the smoke tests)

Checklist

Terms

…heading

That is, with the build in date-grouping facility:
1. Invalid dates are in the first group
2. The field name is now included in the group name, which helps with identifying errors.

This fixes #2591, for built-in date-grouping code.
 #2591 is fixed.
 And I realised that my 'Default sort order' was wrong, as whether tasks are overdue
 affects the sort order.
@claremacrae claremacrae added the scope: grouping Changes to the grouping capabilities label Jan 17, 2024
@claremacrae claremacrae merged commit d07f2dc into main Jan 17, 2024
2 checks passed
@claremacrae claremacrae deleted the fix-group-by-date-undated branch January 17, 2024 22:44
@claremacrae claremacrae changed the title fix: group by [date] now Invalid [date] date as first heading fix: group by [date] now puts Invalid [date] date as first heading Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: grouping Changes to the grouping capabilities
Projects
Status: 🎉 Released
Development

Successfully merging this pull request may close these issues.

group by due and similar should put Invalid date heading before the headings for valid dates.
1 participant