[Important] Switch to using GET instead of POST for GET-only endpoints

[AhmedZaki99]

Issue:

Todoist developers have made a silly decision of removing support for some HTTP verbs for endpoints that accept both GET and POST. So, according to this article, some endpoints will only accept GET and the rest will only accept POST.

It was not explicitly stated in the Sync API Docs which endpoints should use GET, so, I had to look at each of their curl examples and found that the following endpoints accept GET by default:

• projects/get_archived (#get-archived-projects)
• completed/get_all (#get-all-completed-items)
• uploads/get (#get-uploads)
• activity/get (#get-activity-logs)
• backups/get (#get-backups)
• completed/get_stats (#get-productivity-stats) [Not implemented by this package]
• archive/sections (#get-archived-sections) [Not implemented by this package]
• archive/items (#get-completed-items) [Not implemented by this package]
• archive/items_many (#get-completed-items-with-a-list-of-parent-ids) [Not implemented by this package]

Impact:

From the moment Todoist has made that change, all calls to the ItemsService.GetCompletedAsync method fail with the 405 Method Not Allowed, regardless of the Todoist.Net version used.

Fix:

• A GetAsync method was added to both TodoistRestClient and TodoistClienery similar to the PostAsync method.
• Then, each endpoint listed above was updated to use GetAsync instead of PostAsync.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[olsh]

Sorry for the delay.

Thank you again! 👍🏻