obsidian-tasks-group / obsidian-tasks

Task management for the Obsidian knowledge base.
https://publish.obsidian.md/tasks/
MIT License
2.48k stars 232 forks source link

Warning on the notification's page is outdated [confirmed: differences in behaviour need documenting] #2510

Open LeonDixon opened 10 months ago

LeonDixon commented 10 months ago

Please check that this issue hasn't been reported before.

Expected Behavior

There should be no warning about the recurring dates.

Current behaviour

On the notifications page, there is a warning as seen below: image This should not be present.

Steps to reproduce

  1. Navigate to https://publish.obsidian.md/tasks/Advanced/Notifications
  2. Scroll to the bottom

Which Operating Systems are you using?

Obsidian Version

Docs

Tasks Plugin Version

Docs

Checks

Possible solution

Delete in the markdown

claremacrae commented 10 months ago

Hi @LeonDixon,

Thanks. I previously saw that that the issue in Reminders - https://github.com/uphy/obsidian-reminder/issues/93 - had been marked as fixed. But when I looked at the code at the time, I wasn't confident that it had actually fixed the problem issues.

So as I presume you're using Reminders, could you please kindly confirm whether the behaviour is now correct in the Reminders plugin?

  1. When you complete the example task in every month on the last: reliable and safe several times, do you get the same lines as Tasks?
  2. When you complete the example task in every month: if next calculated date does not exist, move new due date earlier several times, do you get the same lines as Tasks?
  3. When you complete the example task in every month on the 31st: skips months with fewer than 31 days several times, do you get the same lines as Tasks?

Please kindly try those out and paste the results in here for confirmation - saying which version of Reminders you were using.

Thanks in advance.

LeonDixon commented 10 months ago

Hi @claremacrae

I've put a pr in with the details :) https://github.com/obsidian-tasks-group/obsidian-tasks/pull/2514

claremacrae commented 10 months ago

Hi @claremacrae

I've put a pr in with the details :) #2514

Thanks, but first we need results to the questions above please.

claremacrae commented 10 months ago

Oh sorry - I misunderstood - please ignore the above...