Skip to content

Commit a901808

Browse files
authored
Merge pull request #1559 from visualize-admin/feat/flags
fix: Set flag depending on host only if flag not already set
2 parents a7b1001 + e5e5f00 commit a901808

1 file changed

Lines changed: 10 additions & 5 deletions

File tree

app/flags/flag.tsx

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -80,16 +80,21 @@ const isVercelPreviewHost = (host: string) => {
8080
};
8181

8282
const initFromHost = (host: string) => {
83+
const setDefaultFlag = (name: FlagName, value: FlagValue) => {
84+
if (flag(name) === undefined) {
85+
flag(name, value);
86+
}
87+
};
8388
if (
8489
host.includes("localhost") ||
8590
host.includes("test.visualize.admin.ch") ||
8691
isVercelPreviewHost(host)
8792
) {
88-
flag("configurator.add-dataset.new", true);
89-
flag("configurator.add-dataset.shared", true);
90-
flag("layouter.dashboard.free-canvas", true);
91-
flag("layouter.dashboard.shared-filters", true);
92-
flag("search.termsets", true);
93+
setDefaultFlag("configurator.add-dataset.new", true);
94+
setDefaultFlag("configurator.add-dataset.shared", true);
95+
setDefaultFlag("layouter.dashboard.free-canvas", true);
96+
setDefaultFlag("layouter.dashboard.shared-filters", true);
97+
setDefaultFlag("search.termsets", true);
9398
}
9499
};
95100

0 commit comments

Comments
 (0)