feat: resovled comments

SigNoz · Jul 5, 2024 · 48ad2c2 · 48ad2c2
1 parent 7862db5
commit 48ad2c2
Show file tree

Hide file tree

Showing 6 changed files with 24 additions and 22 deletions.
diff --git a/frontend/public/locales/en-GB/trace.json b/frontend/public/locales/en-GB/trace.json
@@ -7,6 +7,5 @@
 		"column": "Column",
 		"maxLines": "Max lines per Row",
 		"addColumn": "Add a column"
-	},
-	"trace_loading_placeholder": "Just a bit of patience, just a little bit’s enough ⎯ we’re getting your traces!"
+	}
 }
diff --git a/frontend/public/locales/en/common.json b/frontend/public/locales/en/common.json
@@ -6,5 +6,6 @@
 	"share": "Share",
 	"save": "Save",
 	"edit": "Edit",
-	"logged_in": "Logged In"
+	"logged_in": "Logged In",
+	"pending_data_placeholder": "Just a bit of patience, just a little bit’s enough ⎯ we’re getting your {{dataSource}}!"
 }
diff --git a/frontend/public/locales/en/trace.json b/frontend/public/locales/en/trace.json
@@ -7,6 +7,5 @@
 		"column": "Column",
 		"maxLines": "Max lines per Row",
 		"addColumn": "Add a column"
-	},
-	"trace_loading_placeholder": "Just a bit of patience, just a little bit’s enough ⎯ we’re getting your traces!"
+	}
 }
diff --git a/frontend/src/container/LogsLoading/LogsLoading.tsx b/frontend/src/container/LogsLoading/LogsLoading.tsx
@@ -1,8 +1,11 @@
 import './LogsLoading.styles.scss';
 
 import { Typography } from 'antd';
+import { useTranslation } from 'react-i18next';
+import { DataSource } from 'types/common/queryBuilder';
 
 export function LogsLoading(): JSX.Element {
+	const { t } = useTranslation('common');
 	return (
 		<div className="loading-logs">
 			<div className="loading-logs-content">
@@ -13,8 +16,7 @@ export function LogsLoading(): JSX.Element {
 				/>
 
 				<Typography>
-					Just a bit of patience, just a little bit’s enough ⎯ we’re getting your
-					logs!
+					{t('pending_data_placeholder', { dataSource: DataSource.LOGS })}
 				</Typography>
 			</div>
 		</div>

diff --git a/frontend/src/container/TracesExplorer/ListView/index.tsx b/frontend/src/container/TracesExplorer/ListView/index.tsx
@@ -129,6 +129,12 @@ function ListView({ isFilterApplied }: ListViewProps): JSX.Element {
 		[columns, onDragColumns],
 	);
 
+	const isDataPresent =
+		!isLoading &&
+		!isFetching &&
+		!isError &&
+		transformedQueryTableData.length === 0;
+
 	return (
 		<Container>
 			{transformedQueryTableData.length !== 0 && (
@@ -146,19 +152,11 @@ function ListView({ isFilterApplied }: ListViewProps): JSX.Element {
 				<TracesLoading />
 			)}
 
-			{!isLoading &&
-				!isFetching &&
-				!isError &&
-				!isFilterApplied &&
-				transformedQueryTableData.length === 0 && (
-					<NoLogs dataSource={DataSource.TRACES} />
-				)}
-
-			{!isLoading &&
-				!isFetching &&
-				transformedQueryTableData.length === 0 &&
-				!isError &&
-				isFilterApplied && <EmptyLogsSearch />}
+			{isDataPresent && !isFilterApplied && (
+				<NoLogs dataSource={DataSource.TRACES} />
+			)}
+
+			{isDataPresent && isFilterApplied && <EmptyLogsSearch />}
 
 			{!isError && transformedQueryTableData.length !== 0 && (
 				<ResizeTable

diff --git a/frontend/src/container/TracesExplorer/TraceLoading/TraceLoading.tsx b/frontend/src/container/TracesExplorer/TraceLoading/TraceLoading.tsx
@@ -2,9 +2,10 @@ import './TraceLoading.styles.scss';
 
 import { Typography } from 'antd';
 import { useTranslation } from 'react-i18next';
+import { DataSource } from 'types/common/queryBuilder';
 
 export function TracesLoading(): JSX.Element {
-	const { t } = useTranslation('trace');
+	const { t } = useTranslation('common');
 	return (
 		<div className="loading-traces">
 			<div className="loading-traces-content">
@@ -14,7 +15,9 @@ export function TracesLoading(): JSX.Element {
 					alt="wait-icon"
 				/>
 
-				<Typography>{t('trace_loading_placeholder')}</Typography>
+				<Typography>
+					{t('pending_data_placeholder', { dataSource: DataSource.TRACES })}
+				</Typography>
 			</div>
 		</div>
 	);