fix: Add missing digest migration to resolve schema diff#528
fix: Add missing digest migration to resolve schema diff#528elie222 merged 2 commits intoelie222:mainfrom
Conversation
|
@edulelis is attempting to deploy a commit to the Inbox Zero Team on Vercel. A member of the Team first needs to authorize it. |
WalkthroughA database migration script was added to modify the schema by removing the Changes
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
apps/web/prisma/migrations/20250627111946_update_digest/migration.sql (1)
1-6: Add an explicit “down” migration or at least document rollback stepsThe script is one-way; rolling back will require manual SQL. Either add a companion
DOWNscript or leave a short comment explaining the exactALTER TABLE … ADD COLUMN …sequence needed to restore the column so that future hot-fixes aren’t blocked.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
apps/web/prisma/migrations/20250627111946_update_digest/migration.sql(1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
`apps/web/**`: Install packages only within the 'apps/web' directory, not at the repository root.
apps/web/**: Install packages only within the 'apps/web' directory, not at the repository root.
📄 Source: CodeRabbit Inference Engine (.cursor/rules/installing-packages.mdc)
List of files the instruction was applied to:
apps/web/prisma/migrations/20250627111946_update_digest/migration.sql
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: Secret Detection
- GitHub Check: Jit Security
- GitHub Check: test
🔇 Additional comments (1)
apps/web/prisma/migrations/20250627111946_update_digest/migration.sql (1)
13-15:DROP COLUMNalready cascades indexes – double-check the extraDROP INDEXPostgreSQL drops indexes on a column automatically when the column is removed. Keeping the explicit
DROP INDEXis harmless but redundant and slightly prolongs lock time on large tables. Consider deleting it unless your target DB requires it.
Summary by CodeRabbit