Skip to content

Fix Todoist apps for API v1 - #3267

Open
tlambrou wants to merge 1 commit into
tidbyt:mainfrom
tlambrou:codex/fix-todoist-api-v1
Open

Fix Todoist apps for API v1#3267
tlambrou wants to merge 1 commit into
tidbyt:mainfrom
tlambrou:codex/fix-todoist-api-v1

Conversation

@tlambrou

@tlambrou tlambrou commented Sep 2, 2026

Copy link
Copy Markdown

Closes #3266

Summary

  • migrate Todoist and Todoist Tasks from retired REST v2 calls to /api/v1/tasks/filter with query, limit=200, and cursor-aware response parsing
  • update Todoist authorization and token endpoints while preserving both apps’ encrypted legacy OAuth credentials and existing configuration scope
  • version and hash cache keys, fix empty/hidden-empty/unauthenticated/API-error rendering, and keep the list app to one priority-sorted API page
  • regenerate both preview GIFs using synthetic content only

Validation

  • pixlet check apps/todoist apps/todoisttasks
  • synthetic Pixlet renders for zero, one, many, 200+, hidden-empty, unauthenticated, API-error, priority, and 500-character long-text states
  • git diff --check

@tidbyt

tidbyt Bot commented Sep 2, 2026

Copy link
Copy Markdown

⚠️ The automated review process is experimental and likely has bugs. Please bear with us as we iron out the kinks and enable you to ship changes at high velocity 🚀

Next Steps

Hello! Thank you so much for your change 🤜 🤛 . There are a few things you need to do:

  • Sign the CLA if you haven't already
  • Ensure your build is green! Any problem will display a proposed solution to try out
  • Get a review, either by Tidbyt Bot or by a Tidbyt engineer

Manual Review Required

Hang tight! A Tidbyt engineer will be by shortly to review your change. Here is what they will be looking for:

Test Details
🟡 App Dir Touches files in multiple directories
🟡 Modules Multiple .star files: apps/todoist/todoist.star, apps/todoisttasks/todoisttasks.star
🟡 Original Author The original author (zephyern) does not match the PR author (tlambrou)

@tidbyt-bot

tidbyt-bot commented Sep 2, 2026

Copy link
Copy Markdown

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@tlambrou

tlambrou commented Sep 2, 2026

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

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.

Fix Todoist apps for API v1 and current OAuth endpoints

2 participants