Skip to content

Conversation

@arpandhakal
Copy link
Collaborator

Changes

  • Added a deletedAt clause for unique index constraint on channel sync table.
  • Added a check for deletedAt to be null while updating mapping table.

Testing Criteria

@arpandhakal arpandhakal self-assigned this Feb 3, 2026
@linear
Copy link

linear bot commented Feb 3, 2026

@vercel
Copy link

vercel bot commented Feb 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dropbox-integration Ready Ready Preview, Comment Feb 3, 2026 11:28am

Request Review

Copy link
Collaborator

Choose a reason for hiding this comment

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

Lets use a proper migration name and not autogenerated one with pnpm drizzle-kit generate --name xyzxyz

@@ -0,0 +1,2 @@
DROP INDEX "uq_channel_sync_channel_id_dbx_root_path";
Copy link
Collaborator

Choose a reason for hiding this comment

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

We shouldn't drop and create an index with same name

Copy link
Collaborator

@rrojan rrojan left a comment

Choose a reason for hiding this comment

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

LGTM 🏆

@arpandhakal arpandhakal merged commit 215d683 into main Feb 3, 2026
4 checks passed
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