HeroBlackInk / ultimate-todoist-sync-for-obsidian

GNU General Public License v3.0
145 stars 20 forks source link

Problem with due date #44

Open Adriz1er opened 1 year ago

Adriz1er commented 1 year ago

I don't know if it's a bug, but when I write a task on obsidian with a due date. The deadline is always at 8 o'clock when I don't even want there to be a time on the due date. recording-2023-07-16-16-33-59.webm

HeroBlackInk commented 1 year ago

Due to the API of Todoist using the UTC time zone, all dates must include a time in order to be converted into UTC dates. Therefore, I have set it to 8 AM in the morning.

Adriz1er commented 1 year ago

I search on the internet and I find this page : https://developer.todoist.com/sync/v8/#full-day-dates

Out of curiosity, would it be possible to use this?

HeroBlackInk commented 1 year ago

It is a problem that can be solved, I just opted for a quick solution to save time.

Adriz1er commented 1 year ago

Ok thanks

jettil commented 1 year ago

Out of Curriosity: How did you manage to get these Intellisense like suggestions? I am new to obsidian and trying to integrate it into my workflows in which Todoist plays a big role so this Plugin comes in very handy. But i am struggling with the due dates

YoungJY1 commented 11 months ago

Is there anything going on about the time being fixed to 08:00 on "due date"?

I really hope this issue is resolved.

morficus commented 9 months ago

@HeroBlackInk if you would like, I could work on solving this issue so that there is no default time.

The new task operation on the Todoist API now supports passing either due_date OR due_datetime

Jack-KkKk commented 9 months ago

I would love to see this fixed as well. thanks for making the plugin!

Christian-D-Bock commented 1 month ago

Hey @HeroBlackInk - now that Todoist supports a calendar view for upcoming tasks, it seems essential that tasks allow a user to set a time of day, not defaulting to 8am. Unfortunately, even if you update the time in Todoist directly, or drag it on the calendar in Todoist, it reverts to 8am, preventing any use of the new Todoist calendar feature.

Discussion above gives a few possible avenues, though assume it'd be the one at "https://developer.todoist.com/sync/v9#floating-due-dates-with-time", or the fixed time zone version below it, depending on implementation.

Would love to see it fixed, because the calendar view in Todoist is a great way of working! If you're not able to do this yourself, please share your thoughts on how someone should implement it, and you might get a pull request submitted.

Great integration- and keen to make further use of it.

Christian-D-Bock commented 1 month ago

Quick addition to the above; I've searched for locations where 8am is being hard coded, and provide as a screenshot here. Looks like this is where the adjustment would be made?: image

Christian-D-Bock commented 1 month ago

Also, FYI, it seems there's a similar PR already- not sure if you can make use of that to resolve? https://github.com/HeroBlackInk/ultimate-todoist-sync-for-obsidian/pull/77