todoist: Unable to perform initial sync due to reminders.is_deleted of wrong type

Using todoist-git in AUR.

$ todoist-cli -v
todoist version 0.20.0

I get the following error:

Error: json: cannot unmarshal number into Go struct field .reminders.is_deleted of type bool

Almost smells like an API issue on the Todoist side returning a number instead of a bool?

About this issue

Original URL
State: open
Created a year ago
Comments: 17 (6 by maintainers)

Commits related to this issue

fix(sync.go): update sync to no longer sync is_deleted flag while the api is being funky Commented out IsDeleted field in the Reminder struct as the API is currently being inconsistent. Work around f... — committed to HacDan/todoist by HacDan a year ago

Most upvoted comments

So I finally heard back from Todoist. They have deprecated the v8 sync API and in the v9 sync api, this now returns a boolean instead of an int.

Because of this, the backend library will need to be reworked a bit to accommodate the new API. I’ll begin work on this, but if someone beats me to it, feel free!

HacDan on Jun 2, 2023

I put a ticket in with the Todoist Support Team yesterday. I haven’t heard anything, though.

Regarding commenting out, I don’t think it should have any negative effect on the app.

A quick ripgrep through the code base returned no hits outside of the lib folder for IsDeleted or is_deleted

@opennomad Can you test this build? I understand if you don’t want to. I’ve also uploaded the minor change to GitHub if you’d like to check the code first and/or build it yourself. Nothing funky, just trying to be transparent. I only uploaded an elf 64 bit build. I can upload other builds if need be.

Unfortunately, because I’m not experiencing the issue, I don’t know if this will resolve the issue. I only commented out the IsDeleted field for Reminders.

HacDan on May 10, 2023