Skip to content

Commit

Permalink
Merge pull request #41220 from janicduplessis/@janic/remove-logs
Browse files Browse the repository at this point in the history
Remove report screen logs
  • Loading branch information
carlosmiceli authored May 24, 2024
2 parents 07a90c7 + 7ecfd97 commit f27a7c5
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 37 deletions.
13 changes: 0 additions & 13 deletions src/pages/home/ReportScreen.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ import useThemeStyles from '@hooks/useThemeStyles';
import useViewportOffsetTop from '@hooks/useViewportOffsetTop';
import useWindowDimensions from '@hooks/useWindowDimensions';
import Timing from '@libs/actions/Timing';
import Log from '@libs/Log';
import Navigation from '@libs/Navigation/Navigation';
import clearReportNotifications from '@libs/Notification/clearReportNotifications';
import Performance from '@libs/Performance';
Expand Down Expand Up @@ -648,18 +647,6 @@ function ReportScreen({
);
}

Log.client(
`[ReportScreen] Debug render state - ${JSON.stringify({
reportMetadata,
shouldShowReportActionList,
isLoading,
shouldShowSkeleton,
isCurrentReportLoadedFromOnyx,
reportID: report.reportID,
reportIDFromRoute,
})}`,
);

return (
<ActionListContext.Provider value={actionListValue}>
<ReactionListContext.Provider value={reactionListRef}>
Expand Down
24 changes: 0 additions & 24 deletions src/pages/home/report/ReportActionsView.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ import usePrevious from '@hooks/usePrevious';
import useWindowDimensions from '@hooks/useWindowDimensions';
import DateUtils from '@libs/DateUtils';
import getIsReportFullyVisible from '@libs/getIsReportFullyVisible';
import Log from '@libs/Log';
import type {CentralPaneNavigatorParamList} from '@libs/Navigation/types';
import * as NumberUtils from '@libs/NumberUtils';
import {generateNewRandomInt} from '@libs/NumberUtils';
Expand Down Expand Up @@ -289,17 +288,6 @@ function ReportActionsView({
*/
const loadOlderChats = useCallback(
(force = false) => {
Log.info(
`[ReportActionsView] loadOlderChats ${JSON.stringify({
isOffline: network.isOffline,
isLoadingOlderReportActions,
isLoadingInitialReportActions,
oldestReportActionID: oldestReportAction?.reportActionID,
hasCreatedAction,
isTransactionThread: !isEmptyObject(transactionThreadReport),
})}`,
);

// Only fetch more if we are neither already fetching (so that we don't initiate duplicate requests) nor offline.
if (
!force &&
Expand Down Expand Up @@ -351,18 +339,6 @@ function ReportActionsView({
// and there are fewer than 23 items, indicating we've reached the oldest message.
const isLoadingOlderReportsFirstNeeded = checkIfContentSmallerThanList() && reportActions.length > 23;

Log.info(
`[ReportActionsView] loadNewerChats ${JSON.stringify({
isOffline: network.isOffline,
isLoadingNewerReportActions,
isLoadingInitialReportActions,
newestReportAction: newestReportAction.pendingAction,
firstReportActionID: newestReportAction?.reportActionID,
isLoadingOlderReportsFirstNeeded,
reportActionID,
})}`,
);

if (
!force &&
(!reportActionID ||
Expand Down

0 comments on commit f27a7c5

Please sign in to comment.