-
-
Notifications
You must be signed in to change notification settings - Fork 959
FeatureFlag add createdAt and updatedAt #2880
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
It’s useful to know when they were modified for debugging and auditing Purposes
|
WalkthroughThis pull request adds timestamp tracking to the FeatureFlag entity. A database migration adds two timestamp columns ( Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 📜 Recent review detailsConfiguration used: Repository UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (2)
🧰 Additional context used📓 Path-based instructions (1)internal-packages/database/prisma/migrations/**/*.sql📄 CodeRabbit inference engine (CLAUDE.md)
Files:
🧠 Learnings (5)📓 Common learnings📚 Learning: 2026-01-12T11:01:34.792ZApplied to files:
📚 Learning: 2026-01-12T11:01:34.792ZApplied to files:
📚 Learning: 2025-05-27T19:30:34.004ZApplied to files:
📚 Learning: 2025-11-27T16:26:37.432ZApplied to files:
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (23)
🔇 Additional comments (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. Comment |
PR Review: FeatureFlag add createdAt and updatedAtSummaryThis PR adds Code Quality ✅
Potential Issue
|
It’s useful to know when they were modified for debugging and auditing.
For existing rows createdAt and updatedAt are set to now() during the migration, to avoid a nullable column.