Merge pull request #3704 from prince0xdev/fix/disable-download-when-no-data

Fix: Disable download button when no data available
Fix DownloadButton to avoid duplicate downloadCsv call
2025-12-06 01:18:00 +01:00 · 2025-11-10 22:43:42 -08:00 · 2025-11-10 22:43:22 -08:00 · 2025-11-10 22:42:20 -08:00 · 2025-11-08 22:33:22 +01:00 · 2025-11-08 18:22:38 +01:00
2 changed files with 2 additions and 1 deletions
--- a/.gitignore
+++ b/.gitignore
@ -5,6 +5,7 @@ node_modules
 .pnp
 .pnp.js
 .pnpm-store
+package-lock.json

 # testing
 /coverage
--- a/src/components/input/DownloadButton.tsx
+++ b/src/components/input/DownloadButton.tsx
@ -19,7 +19,7 @@ export function DownloadButton({

  return (
    <TooltipTrigger delay={0}>
-      <Button variant="quiet" onClick={handleClick} isDisabled={!data}>
+      <Button variant="quiet" onClick={handleClick} isDisabled={!data || data.length === 0}>
        <Icon>
          <Download />
        </Icon>
Author	SHA1	Message	Date
Mike Cao	7ac5913c86	Merge pull request #3704 from prince0xdev/fix/disable-download-when-no-data Some checks failed Node.js CI / build (postgresql, 18.18, 10) (push) Has been cancelled Details Fix: Disable download button when no data available	2025-11-10 22:43:42 -08:00
Mike Cao	a6e130ab2e	Fix DownloadButton to avoid duplicate downloadCsv call Removed redundant downloadCsv call from handleClick.	2025-11-10 22:43:22 -08:00
Mike Cao	4fe4bb99b7	Apply suggestion from @greptile-apps[bot] Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>	2025-11-10 22:42:20 -08:00
Prince EKPINSE	b9e90268d1	chore [#3699 ] : fix .gitignore syntax and untrack package-lock.json	2025-11-08 22:33:22 +01:00
Prince EKPINSE	cf31ca20b9	Fix: disable download button when no data available	2025-11-08 18:22:38 +01:00
Prince EKPINSE	c3f3a7db3f	Fix: disable download button when no data available	2025-11-08 18:17:22 +01:00