Repository navigation
[NAE-2487] Add Column Configuration and Sorting to Menu Item - #481
SamuelPalaj wants to merge 1 commit into
Conversation
- added functionality to define which headers are selected for sorting by default - added sort direction field
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (6)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. WalkthroughCase and task tabbed views now configure active header sort columns and sort direction. Their body classes serialize the active columns as comma-separated text and the direction as an enumeration-map value. Both configurations default the direction to descending and include Slovak and German labels. ChangesTabbed view header sorting
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Feature Merge Risk: ⚪ Minimal · up to The backend stores the new sorting settings, but their effect on displayed sorting cannot be confirmed from the available code; no specific merge-blocking impact is established. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)✅ Passed checks (4 passed)Full details: Docstring CoverageExplanation Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 4 files. (2 skipped: 2 unsupported.)
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 |
|



Description
Implements NAE-2487
Test Configuration
Checklist:
Summary by CodeRabbit