content: Shelf has eleven reports, and Top Booked Kits was in none of the lists - #262
Conversation
… the lists Top Booked Kits shipped on 2026-06-18 (shelf.nu 8af9ddbac) and was never added to the report enumerations outside content/features/reports.mdx and the reports KB. Nine pages plus src/data/pricing.features.ts still said ten and listed ten. The numeral is dropped rather than bumped: a hard count in copy is a liability with no reader value, and the enumerated list counts itself.
|
@coderabbitai review |
Preview deployment |
⚡ Perf regression gate
Duration: 43.7s Full Playwright HTML report uploaded as |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (10)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. WalkthroughThe change updates report documentation across pricing data, glossary content, alternative pages, and the dashboard page. It adds “Top Booked Kits,” removes outdated report-count wording, and refreshes page metadata dates. ChangesOperational reports documentation
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to The PR updates report listings and removes a drifting hard-coded count so customer-facing pages accurately describe the available reports; no actionable merge-blocking risk remains beyond normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files. (9 skipped: 9 unsupported.) ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
|
2 participants
Triggered by
Not a PR. Found while checking what
content/features/dashboard.mdxsays about Reports for shelf.nu#2939, which touches the dashboard loaders.What was wrong
apps/webapp/app/modules/reports/registry.tshas eleven entries, allenabled: true. Top Booked Kits shipped on 2026-06-18 in shelf.nu commit8af9ddbac.Two pages picked it up at the time:
content/features/reports.mdxandcontent/knowledge-base/getting-started-with-reports.mdx. Nothing else did. Ten other places still said ten and enumerated the same ten names, so for over two months the comparison pages, the dashboard page, the glossary and the plan-comparison matrix all sold Shelf short by one whole report, on exactly the surfaces a prospect reads when weighing us against a competitor's reporting library.The fix
Two changes everywhere the list appears:
Content changes
Factual corrections
content/features/dashboard.mdx- "ten ready-to-run reports" in the Going Deeper with Reports sectioncontent/glossary/operational-reports.mdx- "ten built-in operational reports", plus a Top booked kits row in the Common Operational Reports tablesrc/data/pricing.features.ts- thereportsrow's description, which is what the plan-comparison matrix renderscontent/alternatives/wasp.mdxcontent/alternatives/hector.mdxcontent/alternatives/webcheckout.mdxcontent/alternatives/brite-check.mdxcontent/alternatives/asset-tiger.mdxcontent/alternatives/gocodes.mdxcontent/alternatives/ezofficeinventory.mdxSkipped, blocked by open PRs
content/alternatives/snipe-it.mdx- already being changed in content: Companion 1.4.0 is live — booking calendar, audit evidence, model and SAM ID #259 and content: a Personal workspace holds one person, and now the site says so #260content/alternatives/spreadsheets.mdx- already being changed in content: Companion 1.4.0 is live — booking calendar, audit evidence, model and SAM ID #259 and content: a Personal workspace holds one person, and now the site says so #260content/alternatives/blue-tally.mdx- already being changed in content: a Personal workspace holds one person, and now the site says so #260All three carry the same "ten ready-to-run reports" sentence and the same ten-name list. Recorded as a follow-up; they get the identical two-part fix in the next run once those PRs merge.
Deliberately untouched
content/updates/reports-v1.mdxsays ten in its description and body. It was true on its publication date - Top Booked Kits landed weeks later. Dated changelog entries are a record of what shipped when, and rewriting one is a policy call rather than a correction.Review notes
The report list is verified against
apps/webapp/app/modules/reports/registry.tsonmain, not against a planning doc: eleven entries, every oneenabled: true, andgetEnabledReports()filters on exactly that flag. The eleven are Booking Compliance, Top Booked Assets, Top Booked Kits, Monthly Booking Trends, Overdue Items, Asset Inventory, Asset Activity Summary, Asset Utilization, Idle Assets, Asset Distribution, Custody Snapshot.No claim about plan gating changed. Reports remains included on every plan.
Impact scope
Generated by Shelf Content Intelligence - PR-Website Sync
Summary by CodeRabbit
New Features
Documentation