Fix: config (Roster tenant_id fillable / RoleManager Roster model)#1
Fix: config (Roster tenant_id fillable / RoleManager Roster model)#1artemsky wants to merge 1 commit intohdaklue:masterfrom
Roster tenant_id fillable / RoleManager Roster model)#1Conversation
📝 WalkthroughWalkthroughThe pull request introduces configuration-driven model resolution by replacing static hardcoded references to the Roster model with dynamic resolution via a new private method. Additionally, the Roster model's fillable fields are now determined dynamically, including a configurable tenant column sourced from configuration. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
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 |
tenant_columnconfig issue in Roster modelSummary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.