Hi

From the API docs the close task and reopen task methods are expecting a post method, but the get method is being used instead, so it is returning a 405 Method Not Allowed error.

I am putting together a quick pull request for this as well.

Regards
Alan

This constant is currently used in, for example, the TodoistSectionsTrait. The FILTER_SANITIZE_STRING is deprecated in PHP 8.1:

Deprecated as of PHP 8.1.0, use htmlspecialchars() instead.

Source: https://www.php.net/manual/en/filter.filters.sanitize.php

But when I look at the documentation of the FILTER_SANITIZE_STRING it does:

Strip tags and HTML-encode double and single quotes, optionally strip or encode special characters.

So the suggested replacement has a different behavior as htmlspecialchars does not strip the tags.

I can create a PR, but want to discuss the solution first. I think in this case a strip_tags might be sufficient, but then the "HTML-encode double and single quotes" is missing. What do you think?

Wondering if you are planning to implement any Sync API endpoints here. e.g. when creating a project with the REST API, one can't set the indent, etc., so subprojects are basically impossible. I assume Todoist will improve the REST API before they release it, but was just curious.

I just stumbled on the following information at https://developer.todoist.com/rest/v1/#overview:

DEPRECATED: This version of the REST API (v1) is deprecated and will be removed on Nov 30. Please refer to the v2 documentation.

This means, this script will break on November 30th. However, I can't promise I'll find the time to fix anything before.

Heads up!

I appreciate any bug report or other contributions, but please remember that this issue tracker is only for the client library.

I do not maintain the API that this library talks to, nor am I affiliated with Todoist at all. 😞 If you have any issues or questions about the API itself, please have a look at the official documentation.

Thanks! 🙌

See https://groups.google.com/a/doist.com/forum/#!topic/todoist-api/dvy_Pqnie-0

the change to the signature of TodoistClient constructor messed with my setup:

I suppress the ssl verification when running in debug mode. This is because when running the app on my local machine, the SSL will always fail and curl returns

cURL error 60: SSL certificate problem: unable to get local issuer certificate (see https://curl.haxx.se/libcurl/c/libcurl-errors.html)

I use it like this:

new TodoistClient($serviceCode->access_token, ['verify' => !env('APP_DEBUG', false)]);

I would like to propose to have the $config = [] from version < 1.0 as the 3rd argument to the constructor.
7e95c16#diff-aec6dcf89dc5fd658ec91e8310e54ad6de5c9f2da70a8c65db615ff38a8b7625L44-R56

Code example on readme.md is wrong.

I get this error:
PHP Fatal error: Uncaught Error: Class 'FabianBeiner\Todoist\Todoist' not found

So the code on readme.md is this:
$Todoist = new FabianBeiner\Todoist\Todoist('YOUR_API_TOKEN');
Should be this:
$Todoist = new FabianBeiner\Todoist\TodoistClient('YOUR_API_TOKEN');

Maybe tweak the readme, otherwise great work and thank you!

Hi @FabianBeiner, thanks for the approval of my PR. Then i'm here to ask if can you create a new tag about this implementation.
Thank you so much :).

Hello.

first of all, thank you for this package! cool stuff :-)

I got to the point where I am implementing the exceptions of the APIs I am using.
But I am now stuck with the problem:

How can I identify if the case where the access_token was invalidated?

For the other APIs I am consuming, I am checking for 401 responses.
What is the recommended way here?

if (!$client->getAllProjects()) {
  removePairing();
}

I was expecting something more in the likes of

try {
    $client->getAllProjects();
} catch (\FabianBeiner\Todoist\ResponseForbidden $e) {
    removePairing();
}