scottlanoue.com / atmosphere-office
Full document, spreadsheet, slideshow, and diagram tooling
0
fork

Configure Feed

Select the types of activity you want to include in your feed.

fix: pass ensureSheet and renderSheetTabs to importXlsx

The multi-sheet XLSX import update added ensureSheet and
setCellDataForSheet as required parameters but the caller in
main.ts was never updated, causing "o is not a function" crash
on any XLSX import.

Scott Lanoue ac41c7f1 c958e5b7

+12
+12
src/sheets/main.ts
··· 1729 1729 importXlsx(file, { 1730 1730 ydoc, 1731 1731 getActiveSheet, 1732 + ensureSheet, 1732 1733 setCellData, 1734 + setCellDataForSheet: (sheetIdx: number, id: string, data: { v?: string | number; f?: string; s?: Record<string, unknown> }) => { 1735 + const sheet = ensureSheet(sheetIdx); 1736 + const cells = sheet.get('cells'); 1737 + if (!cells) return; 1738 + let yCell = cells.get(id); 1739 + if (!yCell) { yCell = new Y.Map(); cells.set(id, yCell); } 1740 + if (data.v !== undefined) yCell.set('v', data.v); 1741 + if (data.f !== undefined) yCell.set('f', data.f); 1742 + if (data.s) { for (const [k, v] of Object.entries(data.s)) yCell.set(k, v); } 1743 + }, 1733 1744 getCells, 1734 1745 renderGrid, 1746 + renderSheetTabs, 1735 1747 showToast, 1736 1748 evalCache, 1737 1749 DEFAULT_ROWS,