Skip to content

display books in inbox with filters based on location#145

Merged
elfkuzco merged 1 commit intocms-upgradefrom
display-staging-books
Feb 6, 2026
Merged

display books in inbox with filters based on location#145
elfkuzco merged 1 commit intocms-upgradefrom
display-staging-books

Conversation

@elfkuzco
Copy link
Copy Markdown
Contributor

@elfkuzco elfkuzco commented Feb 4, 2026

Rationale

This PR enhances the inbox view to support filters for book locations. The combination of has_titles and location_kind filters can be used to show books which have no title and are in quarantine/staging. The PR also refactors the filters to be populated from route (openzim/zimfarm#1609) and make tables to be responsive (openzim/zimfarm#1624)

Screenshot_20260204_164444

Changes

This closes #142

@elfkuzco elfkuzco self-assigned this Feb 4, 2026
@elfkuzco elfkuzco requested a review from benoit74 February 4, 2026 15:57
@elfkuzco elfkuzco linked an issue Feb 4, 2026 that may be closed by this pull request
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.05%. Comparing base (12101c7) to head (227d376).
⚠️ Report is 3 commits behind head on cms-upgrade.

Additional details and impacted files
@@               Coverage Diff               @@
##           cms-upgrade     #145      +/-   ##
===============================================
+ Coverage        77.00%   77.05%   +0.04%     
===============================================
  Files               40       40              
  Lines             1383     1386       +3     
  Branches           119      119              
===============================================
+ Hits              1065     1068       +3     
  Misses             280      280              
  Partials            38       38              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Contributor

@benoit74 benoit74 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (and thank you for the "additional" feat on page history not been kept)

Base automatically changed from auth-with-ory to cms-upgrade February 6, 2026 07:05
@elfkuzco elfkuzco force-pushed the display-staging-books branch from ae45182 to 227d376 Compare February 6, 2026 07:08
@elfkuzco elfkuzco merged commit f41acde into cms-upgrade Feb 6, 2026
4 checks passed
@elfkuzco elfkuzco deleted the display-staging-books branch February 6, 2026 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Display staging books in inbox

2 participants