Skip to content

165 table output for data#170

Merged
InfinityBowman merged 2 commits into
mainfrom
165-table-output-for-data
Dec 26, 2025
Merged

165 table output for data#170
InfinityBowman merged 2 commits into
mainfrom
165-table-output-for-data

Conversation

@InfinityBowman
Copy link
Copy Markdown
Owner

@InfinityBowman InfinityBowman commented Dec 26, 2025

Summary by CodeRabbit

  • New Features
    • Added AMSTAR 2 quality assessment results table to the project overview. The table displays individual study scores and a summary breakdown of result categories (High, Moderate, Low, Critically Low) for completed AMSTAR 2 appraisals.

✏️ Tip: You can customize this high-level summary in your review settings.

@InfinityBowman InfinityBowman linked an issue Dec 26, 2025 that may be closed by this pull request
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
corates b6291b0 Commit Preview URL Dec 26 2025, 08:09 PM

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Dec 26, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

A new AMSTAR2ResultsTable component displays quality assessment results for studies by fetching checklist data, computing scores from reconciled or first-completed checklists, and rendering a summary with per-study scores. The OverviewTab integrates this component, and a reconciledChecklistId field is added to reconciliation data.

Changes

Cohort / File(s) Summary
AMSTAR2 Results Component
packages/web/src/components/project-ui/overview-tab/AMSTAR2ResultsTable.jsx
New component that iterates through studies, fetches checklist data, computes AMSTAR 2 scores (preferring reconciled checklists), memoizes results, and renders a summary with counts/percentages and a results table with ScoreTag. Includes null/undefined guards and fallback messaging for missing appraisals.
Overview Tab Integration
packages/web/src/components/project-ui/overview-tab/OverviewTab.jsx
Imports new AMSTAR2ResultsTable component and replaces static placeholder in the Tables section with the dynamic results table, passing studies and getChecklistData props.
Reconciliation Data Model
packages/web/src/primitives/useProject/sync.js
Adds reconciledChecklistId field to the reconciliation object within buildStudyFromYMap, sourced from reconciliationData or set to null if absent.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Poem

🐰 A table for AMSTAR studies hops to the stage,
With scores reconciled, a data-driven page,
The checklists align, the summaries shine,
Overview Tab blessed with results so fine! ✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch 165-table-output-for-data

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 13c8637 and b6291b0.

📒 Files selected for processing (3)
  • packages/web/src/components/project-ui/overview-tab/AMSTAR2ResultsTable.jsx
  • packages/web/src/components/project-ui/overview-tab/OverviewTab.jsx
  • packages/web/src/primitives/useProject/sync.js

Comment @coderabbitai help to get the list of available commands and usage tips.

@InfinityBowman InfinityBowman merged commit 9fb7063 into main Dec 26, 2025
1 of 2 checks passed
@InfinityBowman InfinityBowman deleted the 165-table-output-for-data branch December 26, 2025 20:51
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.

Table output for data

2 participants