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

feat: Add 'sort by function' #2577

Merged
merged 17 commits into from
Jan 12, 2024
Merged

feat: Add 'sort by function' #2577

merged 17 commits into from
Jan 12, 2024

Conversation

claremacrae
Copy link
Collaborator

Description

Enable sort by function to return a sort key, which is then used for sorting tasks in search results.

Note:

  • The tests (and hence also behaviour) and docs are not yet up to (my) release standards.
  • I'm merging it now so that I can use it in my main vault, to confirm I'm happy with the behaviour, and to try to come up with some more realistic examples for the docs.

Motivation and Context

Partial implementation of #1479.

How has this been tested?

  • Adding more tests
  • Exploratory testing

Screenshots (if appropriate)

Types of changes

Changes visible to users:

  • New feature (prefix: feat - non-breaking change which adds functionality)
  • Documentation (prefix: docs - improvements to any documentation content for users)

Internal changes:

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

Checklist

Terms

@claremacrae claremacrae added scope: scripting Issues to do with custom filters, custom sorting and similar scope: sorting Changes to the sorting capabilities labels Jan 12, 2024
@claremacrae claremacrae merged commit 472ff7c into main Jan 12, 2024
2 checks passed
@claremacrae claremacrae deleted the custom-sorting branch January 12, 2024 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: scripting Issues to do with custom filters, custom sorting and similar scope: sorting Changes to the sorting capabilities
Projects
Status: 🎉 Released
Development

Successfully merging this pull request may close these issues.

1 participant