Skip to content

Conversation

@mdmohsin7
Copy link
Member

  • Automatically send the newly created tasks to connected integration
  • For apple reminders send bg fcm msg to phone and let the app handle it in bg using remote-notification
  • Fix apple reminders permission issue

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces automatic task synchronization with connected integrations. For Apple Reminders, it uses silent push notifications to trigger background creation on the device, including a fix for iOS 17 permission handling. For other integrations, the backend is updated to trigger sync upon task creation. My review focuses on a critical regression in the backend's OAuth token handling and a performance improvement in the iOS implementation. The changes are otherwise well-structured and address the goals of the pull request.

@mdmohsin7 mdmohsin7 requested a review from beastoin January 17, 2026 17:59
@mdmohsin7 mdmohsin7 merged commit 2eced53 into main Jan 18, 2026
1 check passed
@mdmohsin7 mdmohsin7 deleted the auto-sync-tasks branch January 18, 2026 04:08
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.

2 participants