Skip to content

[comp] Production Deploy#2465

Merged
Marfuen merged 2 commits into
releasefrom
main
Apr 6, 2026
Merged

[comp] Production Deploy#2465
Marfuen merged 2 commits into
releasefrom
main

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented Apr 6, 2026

This is an automated pull request to release the candidate branch into production, which will trigger a deployment.
It was created by the [Production PR] action.

github-actions Bot and others added 2 commits April 6, 2026 20:13
…how per-category breakdown (#2464)

The people table was not accounting for HIPAA training in the task
completion totals, so employees appeared fully complete even when HIPAA
training was pending. Also adds individual category counts (Policies,
Training, HIPAA) to the tasks column for easier at-a-glance tracking,
and fixes the overview chart bar to include HIPAA in the completed segment.

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
app (staging) Building Building Apr 6, 2026 9:27pm
comp-framework-editor Ready Ready Preview, Comment Apr 6, 2026 9:27pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
portal (staging) Skipped Skipped Apr 6, 2026 9:27pm

Request Review

@cursor
Copy link
Copy Markdown

cursor Bot commented Apr 6, 2026

PR Summary

Medium Risk
Updates server-side task completion calculations and UI displays, including conditional HIPAA training counting; incorrect totals could misreport compliance progress for orgs with HIPAA enabled.

Overview
Enhances the People list task progress to show a breakdown by Policies, Training, and (when applicable) HIPAA, instead of a single completed/total count.

On the server side, task completion is recalculated into a richer TaskCompletion shape, conditionally adding a HIPAA training task when the org has a HIPAA framework instance and excluding the HIPAA video from the generic training count. The dashboard chart’s completed-task total is also updated to include HIPAA completion.

Reviewed by Cursor Bugbot for commit 8ac2416. Bugbot is set up for automated code reviews on this repo. Configure here.

@Marfuen Marfuen merged commit 002b36e into release Apr 6, 2026
11 of 12 checks passed
@claudfuen
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 3.16.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 8ac2416. Configure here.

tc.completedAt !== null,
)
? 1
: 0;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

HIPAA completion unreachable when training step disabled

Medium Severity

The trainingCompletions query is gated by securityTrainingStepEnabled, but hipaaCompleted is derived from that same trainingCompletions array. When securityTrainingStepEnabled is false, trainingCompletions is empty, so hipaaCompleted will always be 0 — yet totalHipaaTraining is still 1 when hasHipaaFramework is true. This makes HIPAA tasks permanently incomplete in the progress bar. The dashboard in EmployeesOverview.tsx correctly fetches HIPAA completions independently of that flag.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 8ac2416. Configure here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants