Conversation
◈ PR Lens
Architecture 1 component touched across 3 lanes. Data flow No data-flow sequence changed in this PR. View
Tip Run 🪧 More tips
Thanks for using PR Lens! It's built by Coldtea, free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe archive view now counts archived issues in both subgrouped and grouped pagination paths. Both count filters match the archive queryset. ChangesArchived issue pagination
Priority: ➖ Normal Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Bug fix · Severity of issue fixed: Medium Merge Risk: ⚪ Minimal · up to Archived work items remain discoverable through both grouped archive pagination paths, so the change is mergeable. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ 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 |
Description
Fixes #9813.
Archived work items were not appearing in the Archived section when the project was using grouped or sub-grouped views.
The grouped archive pagination was using
archived_at__isnull=Truefor thecount_filter, which counts non-archived work items. This caused the paginator to report zero results for archived groups.This changes the filter to
archived_at__isnull=Falsefor both grouped and sub-grouped archive views.Type of Change
Screenshots and Media (if applicable)
Not applicable.
Test Scenarios
References
Fixes #9813
Summary by CodeRabbit