Skip to content

[comp] Production Deploy#2837

Merged
tofikwest merged 10 commits into
releasefrom
main
May 13, 2026
Merged

[comp] Production Deploy#2837
tofikwest merged 10 commits into
releasefrom
main

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented May 13, 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.


Summary by cubic

Rebuilt the Background Check tab into a task-focused V1 with Order, Attach, and Exempt paths, and refined the completed report to highlight our verification methods. Adds a credits status strip and improves keyboard navigation and copy.

  • New Features

    • V1 page: status strip with credits + plan link; accessible path picker (radio group) with preserved form state.
    • Three flows: Order (invite), Attach (PDF with vendor/date), Exempt (reason + justification, toggle supported).
    • Report updates: methodology banner, per-section method subtitles, “Identity & liveness,” “Public-source research,” and an audit timeline.
    • Scope panel: collapsible “What’s verified” with simplified rows; removed “Full audited report” and “Social media screening.”
    • Cleanup: removed the old wizard, custom upload, and payment dialog; updated tests and labels.
  • Bug Fixes

    • Fixed arrow-key navigation to follow the current selection.
    • Corrected credit display by filtering to background-check subscriptions only.
    • Trimmed name and email before sending invite requests.
    • Aligned V1 tab content with the tab strip and header by removing extra inner padding.

Written for commit ce1564b. Summary will update on new commits.

github-actions Bot and others added 3 commits May 13, 2026 20:44
…verification methodology

Replaces generic benefit bullets ("Required for Compliance",
"Social media verifications") with a marketing-forward methodology block
that names what we actually do: biometric ID + liveness, HR-confirmed
employment, structured reference questionnaires, AI-augmented
public-source research, and FCRA-style adjudication.

The completed report now opens with a methodology banner and each section
shows its verification method as a subtitle. Renames "Social and media
research" to "Public-source research" so LinkedIn stops reading as the
whole product.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
feat(background-check): redesign overview and report to surface real verification methodology
@vercel
Copy link
Copy Markdown

vercel Bot commented May 13, 2026

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

Project Deployment Actions Updated (UTC)
app (staging) Ready Ready Preview, Comment May 13, 2026 11:41pm
comp-framework-editor (staging) Ready Ready Preview, Comment May 13, 2026 11:41pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
portal (staging) Skipped Skipped May 13, 2026 11:41pm

Request Review

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 5 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

tofikwest and others added 5 commits May 13, 2026 18:51
…task surface

Implements the V1 design from `design_handoff_background_check/SPEC.md`:
the Background Check tab's empty state is now a unified task-oriented page
where Order / Attach / Exempt are equal first-class decisions, instead of
a marketing-style benefits grid followed by a separate form and upload UI.

Page anatomy (top to bottom): status strip with credits + plan link → path
picker (3 selectable cards) → contextual body (Order / Attach / Exempt
form per selection, form state preserved across switches) → collapsible
"What's verified in this check" scope panel.

The path picker is a true radiogroup (role=radio, ArrowLeft/Right
navigation, Space/Enter to select). Out-of-credits disables Send invite;
missing PDF disables Attach report; missing reason disables Confirm
exemption.

Wiring:
- Order  → POST /v1/people/:id/background-check (existing endpoint)
- Attach → POST /v1/people/:id/background-check/custom (existing)
- Exempt → PATCH /v1/people/:id with backgroundCheckExempt + reason +
  justification (latter two are new; backend ignores until DTO is
  extended — flagged in PR description)

Removed: BackgroundCheckMethodology, BackgroundCheckWizardParts (Overview
/ BackgroundCheckSummary / BillingCallout), BackgroundCheckDetailsForm,
CustomBackgroundCheckUpload, PaymentMethodUpdateDialog, react-hook-form
glue for the wizard, sessionStorage pending-request stash, and the 402
"out of credits → redirect to billing" flow (V1 just disables the button
and exposes "Choose a plan →" in the status strip).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
… screening" scope rows

Per Lewis: remove both rows from the "What's verified in this check" panel.
The summary count auto-updates from 6 to 4.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Avoids implying a third-party audited report (court records, criminal,
civil) — we don't perform that scope. The check is end-to-end via our
vendor, which is what the description now says.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…r, trim payload

Three correctness fixes flagged by cubic-dev-ai review:

1. Arrow-key path navigation: each PathCard's onNavigate was hardcoding
   its own path id. After the first arrow keypress, selection moved but
   DOM focus stayed on the original card, so subsequent presses computed
   the next path from the stale hardcoded id. Now navigateRelative reads
   selectedPath directly from state, so navigation tracks the current
   selection regardless of where focus lives.

2. computeCredits could pick a non-background-check active subscription
   (e.g. a pentest subscription) and display the wrong credit count.
   Now filters by getBillingSkuProductKey(skuKey) === 'background_check',
   matching how getBackgroundChecksRemaining already selects.

3. Send invite was POSTing employeeName / employeeEmail untrimmed. Now
   trims both before the request body — requesterNotes was already
   trimmed.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
feat(background-check): V1 two-paths redesign — Order / Attach / Exempt
tofikwest and others added 2 commits May 13, 2026 19:35
The V1 page wrapped its content in a 40px-side / 32px-top / 40px-bottom
padded container per SPEC.md, but in our app the parent <TabsContent>
already provides the page-level inset. The extra inner padding pushed
the status strip + path picker inward, so they no longer aligned with
the tab labels (Details, Policies, …) or the H1 above.

Dropping the inner wrapper padding lets the V1 content sit flush with
the tab strip, matching how the other tabs (EmployeeDetails, etc.)
render directly inside their TabsContent.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
fix(background-check): align V1 tab content with tab strip and page header
@vercel vercel Bot temporarily deployed to staging – portal May 13, 2026 23:37 Inactive
@tofikwest tofikwest merged commit 5a69038 into release May 13, 2026
14 checks passed
@claudfuen
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 3.54.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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