Skip to content

Conversation

@goncalossilva
Copy link

@goncalossilva goncalossilva commented Dec 14, 2022

v8 was deprecated on December 5, 2022, and passing tokens as query parameters will stop working soon.

I didn't notice any other incompatibility, but here's the migration guide from v8: https://developer.todoist.com/sync/v9/#migrating-from-v8

v8 was deprecated on December 5, 2022.
Passing tokens as query params will be deprecated soon.
@abhisheknaiidu
Copy link
Owner

@CodiumAI-Agent /review

@CodiumAI-Agent
Copy link

CodiumAI-Agent commented Mar 21, 2025

PR Reviewer Guide 🔍

(Review updated until commit c0695a3)

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

API Migration

Verify that the change to using axios.get with the v9 endpoint and token passed in the header works as expected and maintains compatibility with the previous behavior.

const stats = await axios.get(
  "https://api.todoist.com/sync/v9/completed/get_stats",
  { headers: { Authorization: `Bearer ${TODOIST_API_KEY}` } }

@goncalossilva
Copy link
Author

This PR will still work for now, but note that the Todoist API v1 has just been released which means Sync API v9 will be deprecated in the near future. Migration documentation is available here.

It should still be a small change for this project:

Todoist API 2025-04-16 at 12 47 59 AM

@CodiumAI-Agent
Copy link

Persistent review updated to latest commit c0695a3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants