mirror of
https://github.com/christianvidalwolf-prog/CrazeAnalytix.git
synced 2026-08-03 14:45:24 +02:00
fix: resolve triplicated ASIN in Forecast View and improve metadata mapping
This commit is contained in:
@@ -90,7 +90,7 @@ const App: React.FC = () => {
|
|||||||
console.log('[App] Successfully loaded', data.length, 'rows');
|
console.log('[App] Successfully loaded', data.length, 'rows');
|
||||||
|
|
||||||
// Refresh forecast too
|
// Refresh forecast too
|
||||||
handleForecastFetch(data, globalAsinMetadata);
|
handleForecastFetch(data);
|
||||||
} catch (error) {
|
} catch (error) {
|
||||||
console.error("Failed to fetch/parse CSV", error);
|
console.error("Failed to fetch/parse CSV", error);
|
||||||
alert("Error loading data. Please refresh the page.");
|
alert("Error loading data. Please refresh the page.");
|
||||||
@@ -143,7 +143,7 @@ const App: React.FC = () => {
|
|||||||
}
|
}
|
||||||
}, []);
|
}, []);
|
||||||
|
|
||||||
const handleForecastFetch = useCallback(async (sales: SalesRecord[], meta: Map<string, { sku: string; title: string; line: string }>) => {
|
const handleForecastFetch = useCallback(async (sales: SalesRecord[]) => {
|
||||||
try {
|
try {
|
||||||
console.log('[App] Fetching forecast from /forecast.xlsx...');
|
console.log('[App] Fetching forecast from /forecast.xlsx...');
|
||||||
const response = await fetch('/forecast.xlsx');
|
const response = await fetch('/forecast.xlsx');
|
||||||
@@ -151,6 +151,17 @@ const App: React.FC = () => {
|
|||||||
|
|
||||||
const buffer = await response.arrayBuffer();
|
const buffer = await response.arrayBuffer();
|
||||||
const fcRecords = await processForecastExcel(buffer);
|
const fcRecords = await processForecastExcel(buffer);
|
||||||
|
|
||||||
|
// Build precise metadata map from sales records
|
||||||
|
const meta = new Map<string, { sku: string; title: string; line: string }>();
|
||||||
|
sales.forEach(r => {
|
||||||
|
const asin = r.asin.trim().toUpperCase();
|
||||||
|
const existing = meta.get(asin);
|
||||||
|
if (!existing || (r.title && r.title.length > (existing.title?.length || 0))) {
|
||||||
|
meta.set(asin, { sku: r.sku, title: r.title, line: r.line });
|
||||||
|
}
|
||||||
|
});
|
||||||
|
|
||||||
const viewData = calculateForecastViewData(sales, fcRecords, meta);
|
const viewData = calculateForecastViewData(sales, fcRecords, meta);
|
||||||
setForecastData(viewData);
|
setForecastData(viewData);
|
||||||
console.log('[App] Forecast loaded:', viewData.length, 'records');
|
console.log('[App] Forecast loaded:', viewData.length, 'records');
|
||||||
@@ -234,7 +245,7 @@ const App: React.FC = () => {
|
|||||||
handleTrafficFetch();
|
handleTrafficFetch();
|
||||||
|
|
||||||
// 1d. Fetch Forecast data
|
// 1d. Fetch Forecast data
|
||||||
handleForecastFetch(cachedData || [], globalAsinMetadata);
|
handleForecastFetch(cachedData || []);
|
||||||
};
|
};
|
||||||
initApp();
|
initApp();
|
||||||
}, [handleDataFetch]);
|
}, [handleDataFetch]);
|
||||||
|
|||||||
@@ -1509,7 +1509,10 @@ export const processForecastExcel = async (fileOrBuffer: File | ArrayBuffer): Pr
|
|||||||
|
|
||||||
return jsonData.map(row => ({
|
return jsonData.map(row => ({
|
||||||
asin: String(row['ASIN'] || row['asin'] || '').trim().toUpperCase(),
|
asin: String(row['ASIN'] || row['asin'] || '').trim().toUpperCase(),
|
||||||
annualForecast: parseUnits(String(row['Forecast 2026'] || row['forecast 2026'] || '0'))
|
annualForecast: parseUnits(String(row['Forecast 2026'] || row['forecast 2026'] || '0')),
|
||||||
|
sku: row['SKU'] || row['sku'] || undefined,
|
||||||
|
title: row['Title'] || row['title'] || row['Article Name'] || undefined,
|
||||||
|
line: row['Product Line'] || row['line'] || row['ProductLine'] || undefined
|
||||||
})).filter(r => r.asin && r.annualForecast > 0);
|
})).filter(r => r.asin && r.annualForecast > 0);
|
||||||
} catch (error) {
|
} catch (error) {
|
||||||
console.error("Error processing Forecast Excel:", error);
|
console.error("Error processing Forecast Excel:", error);
|
||||||
@@ -1582,9 +1585,9 @@ export const calculateForecastViewData = (
|
|||||||
|
|
||||||
return {
|
return {
|
||||||
asin: identifier,
|
asin: identifier,
|
||||||
sku: meta?.sku || identifier, // Fallback to ASIN if SKU not found
|
sku: meta?.sku || fc.sku || identifier,
|
||||||
title: meta?.title || identifier,
|
title: meta?.title || fc.title || identifier,
|
||||||
line: meta?.line || "",
|
line: meta?.line || fc.line || "",
|
||||||
annualForecast: fc.annualForecast,
|
annualForecast: fc.annualForecast,
|
||||||
monthlyData
|
monthlyData
|
||||||
};
|
};
|
||||||
|
|||||||
@@ -198,6 +198,9 @@ export interface CombinedKPIs {
|
|||||||
export interface ForecastRecord {
|
export interface ForecastRecord {
|
||||||
asin: string;
|
asin: string;
|
||||||
annualForecast: number;
|
annualForecast: number;
|
||||||
|
sku?: string;
|
||||||
|
title?: string;
|
||||||
|
line?: string;
|
||||||
}
|
}
|
||||||
|
|
||||||
export interface MonthlyForecastPoint {
|
export interface MonthlyForecastPoint {
|
||||||
|
|||||||
Reference in New Issue
Block a user