-
-
Notifications
You must be signed in to change notification settings - Fork 235
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
Native reminder support #1854
Comments
Thanks very much indeed for picking this up @Erik-Handeland. |
Earlier references and requests: |
Yes, this needs to be the approach. In case you haven't seen it, this page will be very useful when adding the field : In the interests of keeping the steps manageable, perhaps the first version could not support reminder times, and could instead have a setting the user says what times of day they would like that day's reminders. |
Tasks is in the fortunate position of PRs coming in very often now, and long-lived PRs have a high chance of conflicting with For example, there are incoming PRs to add support for dataview format, and a new SQL-based search engine. Both of which would overlap with a PR adding a new field. And the larger a PR is, the longer it takes me to review, and again the more potentially conflicting other edits come in. Therefore, ideally, it would really help for this feature to be done in small chunks, for example adding the storage and editing of reminders in one PR and the actual Reminder modal in a different one - at each stage, providing enough documentation and tests that Tasks could be released as soon as the PR is merged. |
Indeed, having too many PRs is a nice problem to have. I had some time to look at the existing reminders plugin and the key functions for notifications seems fairly straight forward (famous last words). I've also taken a look at the two mentions pending PRs and will keep them in mind, hopefully they can be merged in here soon. Love to see all this momentum |
FYI I forked this repo and built this myself. I'd like to properly contribute to this project by making a PR, writing the tests, etc, but have very little bandwidth at the moment. I would really welcome anyone that wants to help get this over the very final lap. Obligatory use at your own risk if you really want to The branch with Reminder support is here: https://github.com/stevendkwtz/obsidian-tasks/tree/develop
You can clone my fork, NOTE that branch contains other changes I've made to this repo, including an API to toggle tasks. Would need to be decoupled from the reminder changeset to be properly merged into this repo. |
Hi @stevendkwtz
Thanks very much for this. Things I've noted from reading through the code, if/when this is picked up:
task.reminderDate?.format(
isDateTime(task.reminderDate)
? TaskRegularExpressions.dateTimeFormat
: TaskRegularExpressions.dateFormat,
),
|
😮 I have not seen this before commenting on this issue even though its linked above.
I was wondering myself whether it made sense for Reminders to be added to happens or not. I added it to recurrence because I assumed that if a user adds a reminder to a recurring task, they'd normally like to be reminded about that task at each recurrence.
Not sure myself 😅. TBH I would defer to you / the other plugin owners and maintainers, any user feedback you may have collected about this, or copy whatever Todoist may do.
No thinking at all and not purposeful, I just hacked this together quick and dirty and can update this. I use this plugin differently than probably every other user, so I built reminders for my use case first, and haven't properly tested or given much thought to most of the other functionality. What I mean by that is:
My personal end goal workflow is tasks+reminder support+global notification support (using ntfy since obsidian doesn't have notifications?!) Let me know what you think some next steps are here, and where the best channel to continue the conversation is as well. |
Hi @stevendkwtz, wow, thanks for a really interesting reply.
Definitely a video call to explore how to move forwards... It's way higher bandwidth than to-and-fro here... My email is in my GitHub profile.... Hope to hear from you... |
We will add support for a Reminder field. But we are not going to implementation a Reminder mechanism in Tasks - that is best done by other plugins or tools. Background: This request falls within a set of areas that we have decided not to support in Tasks. Closing as |
🔖 Feature description
There is user demand for notifications based on task dates see #432. However this requires the use of another obsidian plugin which was not designed to be used with tasks first, instead adding support later in development.
✔️ Solution
Create a native implementation of reminders build into the tasks plugin. See my current ideas for it under Additional Context.
❓ Alternatives
Do nothing, and use the currently existing reminder plugin
📝 Additional Context
Setting Reminder Date
How to set the reminder date:
due, start…
as the reminder date via a selector in settings or a toggle next to the date ⏲️While option one is probably the easiest I think three is the most interesting to implement while also offering the most expandability with multiple reminders and a minimally clutter UI. I do believe a PR should be created to hide elements from the modal see #1853.
The text was updated successfully, but these errors were encountered: