search

Hoffelhas / autodoist

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

Fix parsing of due dates with time #25

Closed RoccoMathijn closed 1 year ago

RoccoMathijn commented 2 years ago

Now it works for both due dates without time (2016-12-16) and due dates with time (2016-12-01T12:00:00)

Fixes https://github.com/Hoffelhas/autodoist/issues/24

Edit: Added some more functionality to take timezones into account

erauner12 commented 2 years ago

Im have -nf set to 3 days. And still having trouble with next action label getting applied with tasks that are due in the future

RoccoMathijn commented 1 year ago

Im have -nf set to 3 days. And still having trouble with next action label getting applied with tasks that are due in the future

Is that with my new proposed code?

erauner12 commented 1 year ago

Yes I was still having issue unfortunately. I can try and re-create to give more info

RoccoMathijn commented 1 year ago

@erauner12 I think your issue is something different than what this PR fixes: https://github.com/Hoffelhas/autodoist/issues/24

This is about start dates not the due dates too far in the future.

The docs talk about parameter -hf while you say -nf. Did you make a typo or is that the problem perhaps?

Hoffelhas commented 1 year ago

Thanks for writing a fix for this. Can't be merged in the latest dev version, but has been included manually.