docs: Clarify protectedFields supports system fields and add test#10286
Conversation
|
🚀 Thanks for opening this pull request! We appreciate your effort in improving the project. Please let us know once your pull request is ready for review. Tip
Note Please respond to review comments from AI agents just like you would to comments from a human reviewer. Let the reviewer resolve their own comments, unless they have reviewed and accepted your commit, or agreed with your explanation for why the feedback was incorrect. Caution Pull requests must be written using an AI agent with human supervision. Pull requests written entirely by a human will likely be rejected, because of lower code quality, higher review effort and the higher risk of introducing bugs. Please note that AI review comments on this pull request alone do not satisfy this requirement. |
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughThe PR updates the documentation for the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~8 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
protectedFields supports system fields and add test
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## alpha #10286 +/- ##
==========================================
- Coverage 92.53% 92.13% -0.41%
==========================================
Files 192 192
Lines 16484 16484
Branches 227 227
==========================================
- Hits 15254 15187 -67
- Misses 1210 1273 +63
- Partials 20 24 +4 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
🎉 This change has been released in version 9.7.0-alpha.1 |
Pull Request
Issue
The
protectedFieldsoption documentation did not clarify that system fields likecreatedAtandupdatedAtcan also be protected. This could lead developers to assume only custom fields are supported.Approach
protectedFieldsoption documentation that any field can be protected, including system fields likecreatedAtandupdatedAt.createdAtandupdatedAtare hidden from non-owners when listed inprotectedFields.Tasks
Summary by CodeRabbit
Release Notes
Documentation
protectedFieldsconfiguration option, including system fields such ascreatedAtandupdatedAt.Tests