_requiredReportTypeIds without considering whether the slot is marked primary-only or whether the job’s charge for that service is the primary charge.RequiredReportType.primaryOnly should only surface when the related charge is primary (isPrimary on the charge); otherwise those slots should be omitted so add-on jobs do not show empty primary-only rows.Attik Mobile
components/inspection/reports/reportGrouping.ts (getServicesWithRequiredReports, getServiceFromCharge, getAllServices, and downstream helpers such as getUnmatchedReportsByService / getTrulyUnmatchedReports). Today getServicesWithRequiredReports includes every populated RequiredReportType with no primaryOnly or charge-primary check.components/inspection/ReportsSection.tsx consumes getServicesWithRequiredReports and related grouping functions from that module; any change to which slots exist flows through this UI.app/(app)/inspection/[id].tsx (includes ReportsSection).inspection.charges and populated _serviceId (including isPrimary on charges and primaryOnly on required report types where populated).Attik Frontend (parity reference)
src/app/tools/inspections/[id]/components/reports/ (Reports.tsx, ReportCard.tsx, useReportUpload.ts). This tree is the closest in-repo reference for how web lists required slots and handles uploads; it does not currently apply a primaryOnly + charge-primary filter in the scanned servicesWithRequiredReports logic, so the exact parity target may be legacy toolsv2 or a shared rule to be confirmed during implementation.attik-mobile/components/inspection/reports/reportGrouping.ts, attik-mobile/components/inspection/ReportsSection.tsx, attik-mobile/app/(app)/inspection/[id].tsxattik-frontend/src/app/tools/inspections/[id]/components/reports/Reports.tsxPlease authenticate to join the conversation.
Completed
Main App
About 2 months ago
Linear
Get notified by email when there are changes.
Completed
Main App
About 2 months ago
Linear
Get notified by email when there are changes.