search

Hoffelhas / autodoist

Added functionality for Todoist.
MIT License
178 stars 24 forks source link

Don't stop processing sections if task is past hide_future #22

Closed obbardc closed 1 year ago

obbardc commented 2 years ago

If the top task in a section has a due date past hide_future, the tasks after it do not get a Next tag. Let's fix that.

Signed-off-by: Christopher Obbard chris@64studio.com

obbardc commented 2 years ago

hm it doesn't work; if there are three cards: one with no date, one with date after hide_future and one with no date, the no date cards both end up with Next. A forth card does not end up with a label though ;-)

I can live with that, i don;t really have time to do a workaround right now, but i guess one way could be to use a counter instead of a boolean; skip the rest of that section once the counter >= 1 ?

obbardc commented 2 years ago

hm it doesn't work; if there are three cards: one with no date, one with date after hide_future and one with no date, the no date cards both end up with Next. A forth card does not end up with a label though ;-)

I can live with that, i don;t really have time to do a workaround right now, but i guess one way could be to use a counter instead of a boolean; skip the rest of that section once the counter >= 1 ?

It got annoying so I fixed it; now all cases seem to work properly with hide_future

Hoffelhas commented 1 year ago

Hi there. Thanks for your support on this. Due to the major overhaul this change can't be merged any more, but in the latest version all tasks will still get a label if a previous (top) task in a section has a due date that's past the hide_future flag.

Closing this one for now.