Compare commits
No commits in common. "819b43bd77cfb4885e9c96372a43fb3715ba1e30" and "c24e13c29ae929ffad2df1085c664e0e06f1b473" have entirely different histories.
819b43bd77
...
c24e13c29a
@ -260,7 +260,7 @@ export const SALES_REPORT_SO_NAMES = [
|
||||
// --- Pipeline Endpoints ---
|
||||
export const PIPELINE = {
|
||||
endpoints: {
|
||||
nearestStores: '/api/p/krishna-group/v1/nearest-stores',
|
||||
nearestStores: '/api/papi2/nearest-stores',
|
||||
dailySalesReport: '/api/papi2/daily-sales-report',
|
||||
productiveCallSummary: '/api/papi2/productive-call-summary',
|
||||
salesOfficers: '/api/papi2/sales-officers'
|
||||
@ -270,6 +270,3 @@ export const PIPELINE = {
|
||||
export const HIDDEN_FORM_FIELDS = [
|
||||
'store_code_3'
|
||||
];
|
||||
|
||||
export const LOOKUP_RECORD_LIMIT = 500;
|
||||
export const MAX_SKUS_PER_CHUNK = 20;
|
||||
|
||||
@ -1,7 +1,7 @@
|
||||
import React, { useState, useEffect, useCallback, useRef } from 'react';
|
||||
import type { ZinoClient } from '../../api/client';
|
||||
import type { FormScreenResponse } from '../../api/types';
|
||||
import { ORDER_BOOKING, PIPELINE, LOOKUP_RECORD_LIMIT } from '../../api/config';
|
||||
import { ORDER_BOOKING, PIPELINE } from '../../api/config';
|
||||
import { Button } from '../buttons/Button';
|
||||
import { Select } from '../reusable/Select';
|
||||
import { DateField, TimeField, FileInput, TextField } from './fields';
|
||||
@ -184,7 +184,7 @@ export function LogVisitForm({ client, onSuccess, onCancel, onActivityChange }:
|
||||
activityId,
|
||||
fieldId: 'select_store',
|
||||
formData: formDataToSend,
|
||||
limit: LOOKUP_RECORD_LIMIT,
|
||||
limit: 200,
|
||||
});
|
||||
|
||||
const arr = Array.isArray(lookupRes)
|
||||
@ -253,7 +253,7 @@ export function LogVisitForm({ client, onSuccess, onCancel, onActivityChange }:
|
||||
activityId,
|
||||
fieldId: 'daily_log',
|
||||
formData: payload,
|
||||
limit: LOOKUP_RECORD_LIMIT,
|
||||
limit: 200,
|
||||
});
|
||||
|
||||
const arr = Array.isArray(dailyLogLookupRes)
|
||||
|
||||
@ -1,6 +1,5 @@
|
||||
import { useState, useEffect } from 'react';
|
||||
import type { ZinoClient } from '../../../api/client';
|
||||
import { LOOKUP_RECORD_LIMIT } from '../../../api/config';
|
||||
import { SelectField } from './SelectField';
|
||||
|
||||
export interface WfLookupFieldProps {
|
||||
@ -32,7 +31,7 @@ export function WfLookupField({ label, required, value, onChange, client, config
|
||||
activityId,
|
||||
fieldId,
|
||||
formData: JSON.parse(formDataStr),
|
||||
limit: LOOKUP_RECORD_LIMIT
|
||||
limit: 200
|
||||
})
|
||||
.then(res => {
|
||||
if (!mounted) return;
|
||||
|
||||
@ -71,6 +71,13 @@ export function DailyLogMap({
|
||||
{
|
||||
latitude: lat,
|
||||
longitude: lng
|
||||
},
|
||||
{
|
||||
'accept': 'application/json, text/plain, */*',
|
||||
'groupid': '25',
|
||||
'orgid': '57',
|
||||
'templateid': '189',
|
||||
'x-pipeline-version': 'latest'
|
||||
}
|
||||
);
|
||||
|
||||
|
||||
@ -102,7 +102,10 @@ export function AnalyticsChart({ data, gridCols }: AnalyticsChartProps) {
|
||||
|
||||
const fullSpanClass = gridCols === 2 ? 'lg:col-span-2' : 'lg:col-span-3';
|
||||
|
||||
const minChartWidth = 100;
|
||||
// Calculate a dynamic width if there's a lot of data to allow horizontal scrolling
|
||||
const minChartWidth = chartType === 0 || chartType === 1 || chartType === 2
|
||||
? Math.max(100, finalData.length * (hasSeries ? seriesKeys.length * 15 + 40 : 60))
|
||||
: 100;
|
||||
|
||||
const renderChartContent = () => {
|
||||
return (
|
||||
@ -122,11 +125,8 @@ export function AnalyticsChart({ data, gridCols }: AnalyticsChartProps) {
|
||||
dataKey="dimension"
|
||||
axisLine={false}
|
||||
tickLine={false}
|
||||
tick={{ fontSize: 11, fill: '#64748B', angle: -45, textAnchor: 'end' }}
|
||||
interval={0}
|
||||
tick={{ fontSize: 12, fill: '#64748B' }}
|
||||
dy={10}
|
||||
dx={-5}
|
||||
height={70}
|
||||
/>
|
||||
<YAxis
|
||||
axisLine={false}
|
||||
@ -190,7 +190,7 @@ export function AnalyticsChart({ data, gridCols }: AnalyticsChartProps) {
|
||||
|
||||
return (
|
||||
<Card key={chart.chart_uid || idx} title={title} className={`shadow-sm border border-gray-100 !bg-[var(--tiles-card-bg)] ${isFullWidth ? fullSpanClass : ''}`}>
|
||||
<div className={`h-[320px] w-full mt-4 ${minChartWidth > 100 ? 'overflow-x-auto scrollbar-slim' : ''}`}>
|
||||
<div className={`h-[320px] w-full mt-4 overflow-y-hidden ${minChartWidth > 100 ? 'overflow-x-auto scrollbar-slim' : ''}`}>
|
||||
{chartType === 5 ? (
|
||||
<div className="w-full h-full overflow-y-auto pr-2 scrollbar-slim">
|
||||
<table className="w-full text-left text-sm text-gray-600 border-collapse">
|
||||
|
||||
@ -77,16 +77,13 @@ export function RecordView({
|
||||
|
||||
const [internalSortBy, setInternalSortBy] = useState<string | undefined>(sortBy);
|
||||
const [internalSortDir, setInternalSortDir] = useState<'asc' | 'desc' | undefined>(sortDir);
|
||||
const [isUserSorted, setIsUserSorted] = useState(false);
|
||||
|
||||
useEffect(() => {
|
||||
setInternalSortBy(sortBy);
|
||||
setInternalSortDir(sortDir);
|
||||
setIsUserSorted(false);
|
||||
}, [sortBy, sortDir]);
|
||||
|
||||
const handleSort = (fieldKey: string) => {
|
||||
setIsUserSorted(true);
|
||||
if (internalSortBy === fieldKey) {
|
||||
if (internalSortDir === 'asc') {
|
||||
setInternalSortDir('desc');
|
||||
@ -193,8 +190,10 @@ export function RecordView({
|
||||
.map((k) => byKey.get(k) ?? ({ field_key: k, output_label: k, data_type: 'string', is_filter: false, is_search: false } as RecordViewField))
|
||||
.filter(Boolean);
|
||||
}
|
||||
const omitSet = new Set([...(omitColumns || []), 'instance_id']);
|
||||
result = result.filter((f) => !omitSet.has(f.field_key));
|
||||
if (omitColumns) {
|
||||
const omitSet = new Set(omitColumns);
|
||||
result = result.filter((f) => !omitSet.has(f.field_key));
|
||||
}
|
||||
return result;
|
||||
}, [resp, columns, omitColumns]);
|
||||
|
||||
@ -369,7 +368,7 @@ export function RecordView({
|
||||
>
|
||||
<div className="flex items-center gap-1.5">
|
||||
{f.output_label}
|
||||
{internalSortBy === f.field_key && isUserSorted && (
|
||||
{internalSortBy === f.field_key && (
|
||||
internalSortDir === 'asc' ? <ChevronUp size={14} className="text-ruby-600" /> : <ChevronDown size={14} className="text-ruby-600" />
|
||||
)}
|
||||
</div>
|
||||
|
||||
@ -79,10 +79,6 @@ export function formatValue(value: unknown, fieldKey?: string): string {
|
||||
if ('business_name_2' in o && o.business_name_2 != null) return String(o.business_name_2);
|
||||
if ('business_name_3' in o && o.business_name_3 != null) return String(o.business_name_3);
|
||||
if ('business_name' in o && o.business_name != null) return String(o.business_name);
|
||||
|
||||
// Location JSON
|
||||
if ('latitude' in o && 'longitude' in o) return `latitude:${o.latitude},longitude:${o.longitude}`;
|
||||
if ('lat' in o && 'lng' in o) return `latitude:${o.lat},longitude:${o.lng}`;
|
||||
}
|
||||
|
||||
try {
|
||||
|
||||
@ -5,8 +5,7 @@ import {
|
||||
SALES_REPORT_ROUTES,
|
||||
ROUTE_WISE_DISTRIBUTORS,
|
||||
BASE_URL,
|
||||
PIPELINE,
|
||||
MAX_SKUS_PER_CHUNK
|
||||
PIPELINE
|
||||
} from '../api/config';
|
||||
import { Download, Printer } from 'lucide-react';
|
||||
import jsPDF from 'jspdf';
|
||||
@ -17,7 +16,7 @@ export function DailySalesReportPage() {
|
||||
const [route, setRoute] = useState('');
|
||||
const [distributor, setDistributor] = useState('');
|
||||
const [soEmail, setSoEmail] = useState('');
|
||||
const [soOptions, setSoOptions] = useState<{ value: string, label: string }[]>([]);
|
||||
const [soOptions, setSoOptions] = useState<{value: string, label: string}[]>([]);
|
||||
|
||||
const [busy, setBusy] = useState(false);
|
||||
const [error, setError] = useState<string | null>(null);
|
||||
@ -73,64 +72,35 @@ export function DailySalesReportPage() {
|
||||
|
||||
// Meta Info
|
||||
doc.setFontSize(10);
|
||||
|
||||
let leftY = 40;
|
||||
if (distributor) {
|
||||
doc.text(`Distributor Name : ${distributor}`, 14, leftY);
|
||||
leftY += 5;
|
||||
}
|
||||
|
||||
let rightY = 40;
|
||||
if (date) {
|
||||
doc.text(`Date : ${date}`, rightMargin, rightY, { align: 'right' });
|
||||
rightY += 5;
|
||||
}
|
||||
|
||||
if (soEmail) {
|
||||
const selectedSo = soOptions.find(o => o.value === soEmail);
|
||||
doc.text(`SO Name : ${selectedSo ? selectedSo.label : soEmail}`, rightMargin, rightY, { align: 'right' });
|
||||
rightY += 5;
|
||||
}
|
||||
doc.text(`Distributor Name : ${distributor}`, 14, 40);
|
||||
doc.text(`Date : ${date}`, rightMargin, 40, { align: 'right' });
|
||||
const selectedSo = soOptions.find(o => o.value === soEmail);
|
||||
doc.text(`SO Name : ${selectedSo ? selectedSo.label : soEmail}`, rightMargin, 45, { align: 'right' });
|
||||
};
|
||||
|
||||
// Split headers into chunks of exactly MAX_SKUS_PER_CHUNK, even if it means splitting a brand.
|
||||
// Split headers into chunks to avoid jspdf-autotable's horizontal page break bugs
|
||||
// This allows us to keep the 2-tier header perfectly intact and break at heading boundaries.
|
||||
const MAX_SKUS_PER_CHUNK = 14;
|
||||
const headerChunks: any[][] = [];
|
||||
let currentChunk: any[] = [];
|
||||
let currentSkuCount = 0;
|
||||
|
||||
reportData?.response?.headers?.forEach((h: any) => {
|
||||
const allSkus = h.skus || [];
|
||||
if (allSkus.length === 0) return;
|
||||
|
||||
let remainingSkus = [...allSkus];
|
||||
|
||||
while (remainingSkus.length > 0) {
|
||||
const availableSpace = MAX_SKUS_PER_CHUNK - currentSkuCount;
|
||||
|
||||
// Take as many SKUs as we can fit in the current chunk
|
||||
const skusToTake = remainingSkus.splice(0, availableSpace);
|
||||
|
||||
currentChunk.push({
|
||||
...h,
|
||||
skus: skusToTake
|
||||
});
|
||||
|
||||
currentSkuCount += skusToTake.length;
|
||||
|
||||
if (currentSkuCount >= MAX_SKUS_PER_CHUNK) {
|
||||
headerChunks.push(currentChunk);
|
||||
currentChunk = [];
|
||||
currentSkuCount = 0;
|
||||
}
|
||||
const skus = h.skus || [];
|
||||
if (currentSkuCount + skus.length > MAX_SKUS_PER_CHUNK && currentSkuCount > 0) {
|
||||
headerChunks.push(currentChunk);
|
||||
currentChunk = [];
|
||||
currentSkuCount = 0;
|
||||
}
|
||||
currentChunk.push(h);
|
||||
currentSkuCount += skus.length;
|
||||
});
|
||||
|
||||
if (currentChunk.length > 0) {
|
||||
headerChunks.push(currentChunk);
|
||||
}
|
||||
|
||||
if (headerChunks.length === 0) {
|
||||
drawPageHeader(); // Fallback if no data
|
||||
drawPageHeader(); // Fallback if no data
|
||||
}
|
||||
|
||||
headerChunks.forEach((chunkHeaders, index) => {
|
||||
@ -191,7 +161,7 @@ export function DailySalesReportPage() {
|
||||
startY: 50,
|
||||
theme: 'grid',
|
||||
horizontalPageBreak: false,
|
||||
styles: { fontSize: 6, textColor: [0, 0, 0], lineColor: [0, 0, 0], lineWidth: 0.2, cellPadding: 1, minCellWidth: 10, halign: 'center', valign: 'middle' },
|
||||
styles: { fontSize: 6, textColor: [0, 0, 0], lineColor: [0, 0, 0], lineWidth: 0.2, cellPadding: 1, minCellWidth: 10 },
|
||||
headStyles: { fillColor: [243, 244, 246], fontStyle: 'bold', halign: 'center' },
|
||||
didParseCell: (data) => {
|
||||
if (data.section === 'body' && reportData?.response?.total_row && data.row.index === body.length - 1) {
|
||||
@ -319,12 +289,10 @@ export function DailySalesReportPage() {
|
||||
</div>
|
||||
|
||||
<div className="flex justify-between items-start text-sm font-bold text-strong mb-2">
|
||||
<div className="flex flex-col gap-1.5">
|
||||
{distributor && <div>Distributor Name : {distributor}</div>}
|
||||
</div>
|
||||
<div>Distributor Name : {distributor}</div>
|
||||
<div className="text-right flex flex-col gap-1.5">
|
||||
{date && <div>Date : {date}</div>}
|
||||
{soEmail && <div>SO Name : {soOptions.find(o => o.value === soEmail)?.label || soEmail}</div>}
|
||||
<div>Date : {date}</div>
|
||||
<div>SO Name : {soOptions.find(o => o.value === soEmail)?.label || soEmail}</div>
|
||||
</div>
|
||||
</div>
|
||||
|
||||
|
||||
Loading…
Reference in New Issue
Block a user