Skip to content

Commit dc1d46a

Browse files
committed
ENG-1484: drop reload prompts — reactivity is unconditional via dual-write
1 parent 0ee94fa commit dc1d46a

2 files changed

Lines changed: 5 additions & 24 deletions

File tree

apps/roam/src/components/settings/AdminPanel.tsx

Lines changed: 2 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ import { setSetting } from "~/utils/extensionSettings";
1818
import {
1919
getFeatureFlag,
2020
setFeatureFlag,
21-
isNewSettingsStoreEnabled,
2221
} from "~/components/settings/utils/accessors";
2322
import {
2423
onSettingChange,
@@ -372,7 +371,6 @@ const FeatureFlagsTab = (): React.ReactElement => {
372371
);
373372
const [isAlertOpen, setIsAlertOpen] = useState(false);
374373
const [isInstructionOpen, setIsInstructionOpen] = useState(false);
375-
const isReactive = isNewSettingsStoreEnabled();
376374

377375
return (
378376
<div className="flex flex-col gap-4 p-4">
@@ -432,24 +430,14 @@ const FeatureFlagsTab = (): React.ReactElement => {
432430

433431
<Alert
434432
isOpen={isInstructionOpen}
435-
onConfirm={() =>
436-
isReactive ? setIsInstructionOpen(false) : window.location.reload()
437-
}
438-
onCancel={isReactive ? undefined : () => setIsInstructionOpen(false)}
439-
confirmButtonText={isReactive ? "OK" : "Reload Graph"}
440-
cancelButtonText={isReactive ? undefined : "Later"}
433+
onConfirm={() => setIsInstructionOpen(false)}
434+
confirmButtonText="OK"
441435
intent={Intent.PRIMARY}
442436
>
443437
<p>
444438
If this is the first time enabling it, you will need to generate and
445439
upload all node embeddings to supabase.
446440
</p>
447-
{!isReactive && (
448-
<p>
449-
Please reload the graph to see the new &apos;Suggestive Mode&apos;
450-
tab.
451-
</p>
452-
)}
453441
<p>
454442
Go to Suggestive Mode{" "}
455443
{"-> Sync Config -> Click on 'Generate & Upload All Node Embeddings'"}

apps/roam/src/components/settings/HomePersonalSettings.tsx

Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -26,10 +26,7 @@ import { setSetting } from "~/utils/extensionSettings";
2626
import { enablePostHog, disablePostHog } from "~/utils/posthog";
2727
import KeyboardShortcutInput from "./KeyboardShortcutInput";
2828
import streamlineStyling from "~/styles/streamlineStyling";
29-
import {
30-
getFeatureFlag,
31-
isNewSettingsStoreEnabled,
32-
} from "~/components/settings/utils/accessors";
29+
import { getFeatureFlag } from "~/components/settings/utils/accessors";
3330
import { PersonalFlagPanel } from "./components/BlockPropSettingPanels";
3431
import posthog from "posthog-js";
3532

@@ -41,16 +38,12 @@ const HomePersonalSettings = ({ onloadArgs }: { onloadArgs: OnloadArgs }) => {
4138
<div className="flex flex-col gap-4 p-1">
4239
<Label>
4340
Personal node menu trigger
44-
<Description
45-
description={`Override the global trigger for the discourse node menu.${isNewSettingsStoreEnabled() ? "" : " Must refresh after editing."}`}
46-
/>
41+
<Description description="Override the global trigger for the discourse node menu." />
4742
<NodeMenuTriggerComponent extensionAPI={extensionAPI} />
4843
</Label>
4944
<Label>
5045
Node search menu trigger
51-
<Description
52-
description={`Set the trigger character for the node search menu.${isNewSettingsStoreEnabled() ? "" : " Must refresh after editing."}`}
53-
/>
46+
<Description description="Set the trigger character for the node search menu." />
5447
<NodeSearchMenuTriggerSetting onloadArgs={onloadArgs} />
5548
</Label>
5649
<KeyboardShortcutInput

0 commit comments

Comments
 (0)