@quillsql/react 2.16.12 → 2.16.14

package/dist/index.cjs

@@ -24491,8 +24491,9 @@ async function getExportData(client, dashboardFilters, reportId, getToken, event
   }) : [];
   const fetchResp = await quillFetch({
     client,
-    task: "item",
+    task: "report",
     metadata: {
+      reportId,
       dashboardItemId: reportId,
       clientId: client.publicKey,
       databaseType: client?.databaseType,
@@ -24503,9 +24504,19 @@ async function getExportData(client, dashboardFilters, reportId, getToken, event
     },
     getToken
   });
-  const resp = await parseFetchResponse(client, "item", fetchResp, getToken);
+  const resp = await parseFetchResponse(client, "report", fetchResp, getToken);
+  const candidateItem = resp && resp?.rows !== void 0 ? resp : resp?.item ?? resp?.report ?? resp?.data ?? resp;
   const cleanedReport = await cleanDashboardItem({
-    item: resp,
+    item: candidateItem ?? {
+      _id: resp?._id ?? reportId,
+      id: resp?.id ?? reportId,
+      name: resp?.name,
+      rows: resp?.rows ?? [],
+      columns: resp?.columns ?? [],
+      fields: resp?.fields ?? [],
+      pivotRows: resp?.pivotRows ?? [],
+      pivotColumns: resp?.pivotColumns ?? []
+    },
     dashboardFilters,
     client,
     customFields,
@@ -24588,10 +24599,14 @@ var useExport = (reportId, {
         flags,
         schemaData?.customFields
       );
+      const exportRows = resp?.pivot && resp?.pivotRows && downloadOptions?.usePivotRows ? resp.pivotRows : resp?.rows;
+      const exportFields = resp?.pivot && resp?.pivotColumns && downloadOptions?.usePivotRows ? resp.pivotColumns : resp?.columns;
+      const rowsToExport = exportRows && exportRows.length > 0 ? exportRows : dashboardReport?.rows ?? dashboardReport?.pivotRows ?? [];
+      const fieldsToExport = exportFields && exportFields.length > 0 ? exportFields : dashboardReport?.columns ?? dashboardReport?.pivotColumns ?? [];
       downloadCSV({
-        rows: resp.pivot && resp.pivotRows && downloadOptions?.usePivotRows ? resp.pivotRows : resp.rows,
-        fields: resp.pivot && resp.pivotColumns && downloadOptions?.usePivotRows ? resp.pivotColumns : resp.columns,
-        name: resp.name
+        rows: rowsToExport ?? [],
+        fields: fieldsToExport,
+        name: resp?.name || dashboardReport?.name || reports?.[reportId]?.name || "report"
       });
       setIsCSVLoading(false);
     },
@@ -53450,12 +53465,15 @@ var CHART_TYPE_STYLES = {
   table: { height: "700px", width: "1000px" }
 };
 var DEFAULT_STYLE = { height: "400px", width: "600px" };
-function StaticChart({
-  reportId,
-  onClickChartElement,
-  containerStyle,
-  showLegend = false
-}) {
+var MIN_CHART_HEIGHT_PX = 240;
+function StaticChart(props) {
+  const {
+    reportId,
+    onClickChartElement,
+    containerStyle,
+    showLegend = false,
+    className
+  } = props;
   const {
     report,
     loading,
@@ -53465,9 +53483,15 @@ function StaticChart({
     prevPage,
     sortRows
   } = useDashboardReportInternal(reportId);
-  const mergedStyle = {
-    ...report?.chartType ? CHART_TYPE_STYLES[report.chartType] || DEFAULT_STYLE : DEFAULT_STYLE,
-    ...containerStyle
+  const baseStyle = report?.chartType && CHART_TYPE_STYLES[report.chartType] ? CHART_TYPE_STYLES[report.chartType] : DEFAULT_STYLE;
+  const mergedStyle = className ? { ...containerStyle ?? {} } : { ...baseStyle, ...containerStyle ?? {} };
+  const safeContainerStyle = {
+    ...mergedStyle,
+    // Avoid enforcing a fallback minHeight when a className is meant to set
+    // the sizing via CSS. Only apply the safety net when className is absent.
+    ...!className && {
+      minHeight: mergedStyle.minHeight ?? (typeof mergedStyle.height === "number" && mergedStyle.height > 0 ? mergedStyle.height : typeof mergedStyle.height === "string" ? mergedStyle.height.trim().endsWith("%") ? `${MIN_CHART_HEIGHT_PX}px` : mergedStyle.height : `${MIN_CHART_HEIGHT_PX}px`)
+    }
   };
   const onPageChange = (page) => {
     if (page == pageNumber - 1) {
@@ -53492,7 +53516,8 @@ function StaticChart({
       } : void 0,
       onClickChartElement,
       loading,
-      containerStyle: mergedStyle,
+      className,
+      containerStyle: safeContainerStyle,
       showLegend,
       onPageChange,
       tableRowsLoading: pageLoading,

package/dist/index.d.cts

@@ -2503,13 +2503,22 @@ interface ChartEditorProps {
  */
 declare function ChartEditor({ isOpen, reportId, isAdmin, chartBuilderTitle, chartBuilderButtonLabel, onAddToDashboardComplete, onSubmitEditReport, onDiscardChanges, destinationDashboard, destinationSection, organizationName, isHorizontalView, onDelete, setIsOpen, SelectComponent, TextInputComponent, ButtonComponent, SecondaryButtonComponent, HeaderComponent, SubHeaderComponent, LabelComponent, TextComponent, DeleteButtonComponent, ModalComponent, CardComponent, PopoverComponent, LoadingComponent, TableComponent, ChartBuilderInputRowContainer, ChartBuilderInputColumnContainer, PivotRowContainer, PivotColumnContainer, ErrorMessageComponent, ChartBuilderFormContainer, ErrorComponent, hideDeleteButton, hideSubmitButton, showTableFormatOptions, showDashboardFilterFields, chartBuilderOptions, onClickChartElement, onClickChartError, }: ChartEditorProps): react_jsx_runtime.JSX.Element;
 
-interface StaticChartProps {
+interface StaticChartBaseProps {
     reportId: string;
     onClickChartElement?: (data: any) => void;
-    containerStyle?: React.CSSProperties;
     showLegend?: boolean;
 }
-declare function StaticChart({ reportId, onClickChartElement, containerStyle, showLegend, }: StaticChartProps): react_jsx_runtime.JSX.Element;
+type StaticChartProps = (StaticChartBaseProps & {
+    className: string;
+    containerStyle?: never;
+}) | (StaticChartBaseProps & {
+    containerStyle: React.CSSProperties;
+    className?: never;
+}) | (StaticChartBaseProps & {
+    className?: undefined;
+    containerStyle?: undefined;
+});
+declare function StaticChart(props: StaticChartProps): react_jsx_runtime.JSX.Element;
 
 declare const quillFormat: ({ value, format, }: {
     value: any;

package/dist/index.d.ts

@@ -2503,13 +2503,22 @@ interface ChartEditorProps {
  */
 declare function ChartEditor({ isOpen, reportId, isAdmin, chartBuilderTitle, chartBuilderButtonLabel, onAddToDashboardComplete, onSubmitEditReport, onDiscardChanges, destinationDashboard, destinationSection, organizationName, isHorizontalView, onDelete, setIsOpen, SelectComponent, TextInputComponent, ButtonComponent, SecondaryButtonComponent, HeaderComponent, SubHeaderComponent, LabelComponent, TextComponent, DeleteButtonComponent, ModalComponent, CardComponent, PopoverComponent, LoadingComponent, TableComponent, ChartBuilderInputRowContainer, ChartBuilderInputColumnContainer, PivotRowContainer, PivotColumnContainer, ErrorMessageComponent, ChartBuilderFormContainer, ErrorComponent, hideDeleteButton, hideSubmitButton, showTableFormatOptions, showDashboardFilterFields, chartBuilderOptions, onClickChartElement, onClickChartError, }: ChartEditorProps): react_jsx_runtime.JSX.Element;
 
-interface StaticChartProps {
+interface StaticChartBaseProps {
     reportId: string;
     onClickChartElement?: (data: any) => void;
-    containerStyle?: React.CSSProperties;
     showLegend?: boolean;
 }
-declare function StaticChart({ reportId, onClickChartElement, containerStyle, showLegend, }: StaticChartProps): react_jsx_runtime.JSX.Element;
+type StaticChartProps = (StaticChartBaseProps & {
+    className: string;
+    containerStyle?: never;
+}) | (StaticChartBaseProps & {
+    containerStyle: React.CSSProperties;
+    className?: never;
+}) | (StaticChartBaseProps & {
+    className?: undefined;
+    containerStyle?: undefined;
+});
+declare function StaticChart(props: StaticChartProps): react_jsx_runtime.JSX.Element;
 
 declare const quillFormat: ({ value, format, }: {
     value: any;

package/dist/index.js

@@ -24511,8 +24511,9 @@ async function getExportData(client, dashboardFilters, reportId, getToken, event
   }) : [];
   const fetchResp = await quillFetch({
     client,
-    task: "item",
+    task: "report",
     metadata: {
+      reportId,
       dashboardItemId: reportId,
       clientId: client.publicKey,
       databaseType: client?.databaseType,
@@ -24523,9 +24524,19 @@ async function getExportData(client, dashboardFilters, reportId, getToken, event
     },
     getToken
   });
-  const resp = await parseFetchResponse(client, "item", fetchResp, getToken);
+  const resp = await parseFetchResponse(client, "report", fetchResp, getToken);
+  const candidateItem = resp && resp?.rows !== void 0 ? resp : resp?.item ?? resp?.report ?? resp?.data ?? resp;
   const cleanedReport = await cleanDashboardItem({
-    item: resp,
+    item: candidateItem ?? {
+      _id: resp?._id ?? reportId,
+      id: resp?.id ?? reportId,
+      name: resp?.name,
+      rows: resp?.rows ?? [],
+      columns: resp?.columns ?? [],
+      fields: resp?.fields ?? [],
+      pivotRows: resp?.pivotRows ?? [],
+      pivotColumns: resp?.pivotColumns ?? []
+    },
     dashboardFilters,
     client,
     customFields,
@@ -24608,10 +24619,14 @@ var useExport = (reportId, {
         flags,
         schemaData?.customFields
       );
+      const exportRows = resp?.pivot && resp?.pivotRows && downloadOptions?.usePivotRows ? resp.pivotRows : resp?.rows;
+      const exportFields = resp?.pivot && resp?.pivotColumns && downloadOptions?.usePivotRows ? resp.pivotColumns : resp?.columns;
+      const rowsToExport = exportRows && exportRows.length > 0 ? exportRows : dashboardReport?.rows ?? dashboardReport?.pivotRows ?? [];
+      const fieldsToExport = exportFields && exportFields.length > 0 ? exportFields : dashboardReport?.columns ?? dashboardReport?.pivotColumns ?? [];
       downloadCSV({
-        rows: resp.pivot && resp.pivotRows && downloadOptions?.usePivotRows ? resp.pivotRows : resp.rows,
-        fields: resp.pivot && resp.pivotColumns && downloadOptions?.usePivotRows ? resp.pivotColumns : resp.columns,
-        name: resp.name
+        rows: rowsToExport ?? [],
+        fields: fieldsToExport,
+        name: resp?.name || dashboardReport?.name || reports?.[reportId]?.name || "report"
       });
       setIsCSVLoading(false);
     },
@@ -53643,12 +53658,15 @@ var CHART_TYPE_STYLES = {
   table: { height: "700px", width: "1000px" }
 };
 var DEFAULT_STYLE = { height: "400px", width: "600px" };
-function StaticChart({
-  reportId,
-  onClickChartElement,
-  containerStyle,
-  showLegend = false
-}) {
+var MIN_CHART_HEIGHT_PX = 240;
+function StaticChart(props) {
+  const {
+    reportId,
+    onClickChartElement,
+    containerStyle,
+    showLegend = false,
+    className
+  } = props;
   const {
     report,
     loading,
@@ -53658,9 +53676,15 @@ function StaticChart({
     prevPage,
     sortRows
   } = useDashboardReportInternal(reportId);
-  const mergedStyle = {
-    ...report?.chartType ? CHART_TYPE_STYLES[report.chartType] || DEFAULT_STYLE : DEFAULT_STYLE,
-    ...containerStyle
+  const baseStyle = report?.chartType && CHART_TYPE_STYLES[report.chartType] ? CHART_TYPE_STYLES[report.chartType] : DEFAULT_STYLE;
+  const mergedStyle = className ? { ...containerStyle ?? {} } : { ...baseStyle, ...containerStyle ?? {} };
+  const safeContainerStyle = {
+    ...mergedStyle,
+    // Avoid enforcing a fallback minHeight when a className is meant to set
+    // the sizing via CSS. Only apply the safety net when className is absent.
+    ...!className && {
+      minHeight: mergedStyle.minHeight ?? (typeof mergedStyle.height === "number" && mergedStyle.height > 0 ? mergedStyle.height : typeof mergedStyle.height === "string" ? mergedStyle.height.trim().endsWith("%") ? `${MIN_CHART_HEIGHT_PX}px` : mergedStyle.height : `${MIN_CHART_HEIGHT_PX}px`)
+    }
   };
   const onPageChange = (page) => {
     if (page == pageNumber - 1) {
@@ -53685,7 +53709,8 @@ function StaticChart({
       } : void 0,
       onClickChartElement,
       loading,
-      containerStyle: mergedStyle,
+      className,
+      containerStyle: safeContainerStyle,
       showLegend,
       onPageChange,
       tableRowsLoading: pageLoading,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quillsql/react",
-  "version": "2.16.12",
+  "version": "2.16.14",
   "type": "module",
   "exports": {
     ".": {