From 581fa9657adf80733aeaa30579d261d0c6d5f5f0 Mon Sep 17 00:00:00 2001 From: Eli Bosley Date: Fri, 26 Dec 2025 21:20:14 -0500 Subject: [PATCH] fix(onboarding): ensure safe handling of activation onboarding state - Updated the `OnboardingTracker` class to include checks for `activationOnboarding` before building the override snapshot, preventing potential errors when the state is undefined. - This change enhances the robustness of the onboarding process by ensuring that only valid states are processed, improving overall stability. --- api/src/unraid-api/config/onboarding-tracker.module.ts | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/api/src/unraid-api/config/onboarding-tracker.module.ts b/api/src/unraid-api/config/onboarding-tracker.module.ts index 429681c62..0ed02867c 100644 --- a/api/src/unraid-api/config/onboarding-tracker.module.ts +++ b/api/src/unraid-api/config/onboarding-tracker.module.ts @@ -203,7 +203,9 @@ export class OnboardingTracker implements OnApplicationBootstrap, OnApplicationS if (overrideState?.activationOnboarding) { const updatedOverride = this.markOverrideStepCompleted(overrideState, stepId); this.onboardingOverrides.setState(updatedOverride); - return this.buildOverrideSnapshot(updatedOverride.activationOnboarding); + if (updatedOverride.activationOnboarding) { + return this.buildOverrideSnapshot(updatedOverride.activationOnboarding); + } } const currentVersion = @@ -261,7 +263,9 @@ export class OnboardingTracker implements OnApplicationBootstrap, OnApplicationS if (overrideState?.activationOnboarding) { const updatedOverride = this.resetOverrideSteps(overrideState); this.onboardingOverrides.setState(updatedOverride); - return this.buildOverrideSnapshot(updatedOverride.activationOnboarding); + if (updatedOverride.activationOnboarding) { + return this.buildOverrideSnapshot(updatedOverride.activationOnboarding); + } } await this.ensureStateLoaded();