Skip to content

Commit 745485e

Browse files
committed
fix: Limit to including node modules
https://harperdb.atlassian.net/browse/STUDIO-536
1 parent 632838e commit 745485e

File tree

1 file changed

+8
-24
lines changed

1 file changed

+8
-24
lines changed

src/features/instance/applications/modals/DownloadApplicationModal.tsx

Lines changed: 8 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -18,14 +18,10 @@ export function DownloadApplicationModal() {
1818
const { mutate: packageComponent, isPending, isSuccess } = usePackageComponentMutation();
1919
const actionStatus = isSuccess ? `Downloaded` : isPending ? `Downloading` : 'Download';
2020

21-
const [skipNodeModules, setSkipNodeModules] = useState(true);
22-
const [skipSymlinks, setSkipSymlinks] = useState(true);
21+
const [includeNodeModules, setIncludeNodeModules] = useState(false);
2322

24-
const skipNodeModulesChanged = useCallback((e: ChangeEvent<HTMLInputElement>) => {
25-
setSkipNodeModules(e.target.checked);
26-
}, []);
27-
const skipSymlinksChanged = useCallback((e: ChangeEvent<HTMLInputElement>) => {
28-
setSkipSymlinks(e.target.checked);
23+
const includeNodeModulesChanged = useCallback((e: ChangeEvent<HTMLInputElement>) => {
24+
setIncludeNodeModules(e.target.checked);
2925
}, []);
3026

3127
const closeModal = useSetWatchedValue('ShowDownloadApplicationModal', false);
@@ -40,8 +36,7 @@ export function DownloadApplicationModal() {
4036
{
4137
packageName: openedEntry.package,
4238
project: openedEntry.project,
43-
skipNodeModules,
44-
skipSymlinks,
39+
skipNodeModules: !includeNodeModules,
4540
...instanceParams,
4641
},
4742
{
@@ -60,7 +55,7 @@ export function DownloadApplicationModal() {
6055
},
6156
},
6257
);
63-
}, [openedEntry, packageComponent, skipNodeModules, skipSymlinks, instanceParams]);
58+
}, [openedEntry, packageComponent, includeNodeModules, instanceParams]);
6459

6560

6661
return (
@@ -78,21 +73,10 @@ export function DownloadApplicationModal() {
7873
type="checkbox"
7974
className="w-6"
8075
disabled={isPending}
81-
checked={skipNodeModules}
82-
onChange={skipNodeModulesChanged}
83-
/>
84-
<span className="pl-4 pr-8 flex-1 py-2.5">Skip Node Modules</span>
85-
</Label>
86-
87-
<Label className="flex">
88-
<Input
89-
type="checkbox"
90-
className="w-6"
91-
disabled={isPending}
92-
checked={skipSymlinks}
93-
onChange={skipSymlinksChanged}
76+
checked={includeNodeModules}
77+
onChange={includeNodeModulesChanged}
9478
/>
95-
<span className="pl-4 pr-8 flex-1 py-2.5">Skip Symlinks</span>
79+
<span className="pl-4 pr-8 flex-1 py-2.5">Include Node Modules</span>
9680
</Label>
9781

9882
<div className="flex w-full gap-4">

0 commit comments

Comments
 (0)