Skip to content

Commit

Permalink
fix lint and apply suggestions
Browse files Browse the repository at this point in the history
  • Loading branch information
war-in committed Feb 6, 2025
1 parent 1ea90a1 commit 6c504ce
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 5 deletions.
4 changes: 3 additions & 1 deletion src/components/InitialURLContextProvider.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import React, {createContext, useEffect, useMemo, useState} from 'react';
import type {ReactNode} from 'react';
import {Linking} from 'react-native';
import {signInAfterTransitionFromOldDot} from '@libs/actions/Session';
import {AppProps} from '@src/App';
import type {AppProps} from '@src/App';
import CONST from '@src/CONST';
import type {Route} from '@src/ROUTES';
import {useSplashScreenStateContext} from '@src/SplashScreenStateContext';
Expand Down Expand Up @@ -41,6 +41,8 @@ function InitialURLContextProvider({children, url, hybridAppSettings, timestamp}
Linking.getInitialURL().then((initURL) => {
setInitialURL(initURL as Route);
});
// Rules are disabled because changes to splashScreenState could trigger code
// that should only execute when new props (url, hybridAppSettings, timestamp) are passed.
// eslint-disable-next-line react-compiler/react-compiler, react-hooks/exhaustive-deps
}, [url, hybridAppSettings, timestamp]);

Expand Down
4 changes: 2 additions & 2 deletions src/libs/actions/Welcome/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ import type {OnboardingPurpose} from '@src/types/onyx';
import type Onboarding from '@src/types/onyx/Onboarding';
import type TryNewDot from '@src/types/onyx/TryNewDot';
import {isEmptyObject} from '@src/types/utils/EmptyObject';
import * as OnboardingFlow from './OnboardingFlow';
import {clearInitialPath} from './OnboardingFlow';

type OnboardingData = Onboarding | undefined;

Expand Down Expand Up @@ -183,7 +183,7 @@ function resetAllChecks() {
});
isLoadingReportData = true;
isOnboardingInProgress = false;
OnboardingFlow.clearInitialPath();
clearInitialPath();
}

function setSelfTourViewed(shouldUpdateOnyxDataOnlyLocally = false) {
Expand Down
4 changes: 2 additions & 2 deletions src/pages/ErrorPage/SessionExpiredPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import useLocalize from '@hooks/useLocalize';
import useTheme from '@hooks/useTheme';
import useThemeStyles from '@hooks/useThemeStyles';
import Navigation from '@libs/Navigation/Navigation';
import * as Session from '@userActions/Session';
import {clearSignInData} from '@userActions/Session';

function SessionExpiredPage() {
const styles = useThemeStyles();
Expand All @@ -33,7 +33,7 @@ function SessionExpiredPage() {
<TextLink
onPress={() => {
if (!NativeModules.HybridAppModule) {
Session.clearSignInData();
clearSignInData();
Navigation.goBack();
return;
}
Expand Down

0 comments on commit 6c504ce

Please sign in to comment.