Skip to content

Commit d666152

Browse files
authored
fix(settings): use replace navigation to skip settings route when viewing usage (#3240)
1 parent aaa203e commit d666152

2 files changed

Lines changed: 6 additions & 6 deletions

File tree

packages/ui/src/features/settings/sections/PlanUsageSettings.tsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ import { useSeatStore } from "@posthog/ui/features/billing/seatStore";
1313
import { UsageMeter } from "@posthog/ui/features/billing/UsageMeter";
1414
import { useSeat } from "@posthog/ui/features/billing/useSeat";
1515
import { useUsage } from "@posthog/ui/features/billing/useUsage";
16-
import { closeSettings } from "@posthog/ui/features/settings/hooks/useOpenSettings";
16+
import { useSettingsPageStore } from "@posthog/ui/features/settings/stores/settingsPageStore";
1717
import { useSpendAnalysisEnabled } from "@posthog/ui/features/usage/useSpendAnalysisEnabled";
1818
import { navigateToUsage } from "@posthog/ui/router/navigationBridge";
1919
import { track } from "@posthog/ui/shell/analytics";
@@ -329,9 +329,9 @@ export function PlanUsageSettings() {
329329
size="1"
330330
variant="ghost"
331331
onClick={() => {
332-
// Leave settings first so back from /usage skips the settings route.
333-
closeSettings();
334-
navigateToUsage();
332+
// Replace the settings route so back from /usage skips it.
333+
useSettingsPageStore.getState().reset();
334+
navigateToUsage({ replace: true });
335335
}}
336336
>
337337
View usage & spend analysis

packages/ui/src/router/navigationBridge.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -185,8 +185,8 @@ export function navigateToMcpServers(): void {
185185
void getRouterOrNull()?.navigate({ to: "/mcp-servers" });
186186
}
187187

188-
export function navigateToUsage(): void {
189-
void getRouterOrNull()?.navigate({ to: "/usage" });
188+
export function navigateToUsage(options?: { replace?: boolean }): void {
189+
void getRouterOrNull()?.navigate({ to: "/usage", replace: options?.replace });
190190
}
191191

192192
// Channels-space mirrors. These render the same shared views as their /code (or

0 commit comments

Comments
 (0)