Skip to content

Commit a70ca8f

Browse files
samsharabarshathakuri
authored andcommitted
fix: consistent surge alert table
1 parent 66ac866 commit a70ca8f

File tree

4 files changed

+29
-14
lines changed

4 files changed

+29
-14
lines changed

src/views/AllSurgeAlerts/index.tsx

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -194,17 +194,16 @@ export function Component() {
194194
(item) => {
195195
const startDate = isDefined(item.start) ? new Date(item.start) : undefined;
196196
const endDate = isDefined(item.end) ? new Date(item.end) : undefined;
197-
const nowMs = new Date().getTime();
198197

199198
const closed = isDefined(item.end)
200-
? new Date(item.end).getTime() < nowMs : undefined;
199+
? new Date(item.end).getTime() < nowTimestamp : undefined;
201200

202201
if (isDefined(endDate) && closed) {
203202
return endDate.toLocaleString();
204203
}
205204

206205
if (isDefined(startDate)) {
207-
const dateStarted = startDate.getTime() < nowMs
206+
const dateStarted = startDate.getTime() < nowTimestamp
208207
? strings.surgeAlertImmediately
209208
: startDate.toLocaleString();
210209

src/views/EmergencySurge/SurgeTable/index.tsx

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -124,17 +124,16 @@ export default function SurgeTable(props: Props) {
124124
(item) => {
125125
const startDate = isDefined(item.start) ? new Date(item.start) : undefined;
126126
const endDate = isDefined(item.end) ? new Date(item.end) : undefined;
127-
const nowMs = new Date().getTime();
128127

129128
const closed = isDefined(item.end)
130-
? new Date(item.end).getTime() < today : undefined;
129+
? new Date(item.end).getTime() < nowTimestamp : undefined;
131130

132131
if (isDefined(endDate) && closed) {
133132
return endDate.toLocaleString();
134133
}
135134

136135
if (isDefined(startDate)) {
137-
const dateStarted = startDate.getTime() < nowMs
136+
const dateStarted = startDate.getTime() < nowTimestamp
138137
? strings.emergencySurgeImmediately
139138
: startDate.toLocaleString();
140139

src/views/SurgeOverview/SurgeAlertsTable/i18n.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
"surgeAlertsTableEmergency": "Emergency",
1414
"surgeAlertsTableCountry": "Country",
1515
"surgeAlertsTableStatus": "Status",
16-
"surgeAlertsViewAll": "View All Surge Alerts"
16+
"surgeAlertsViewAll": "View All Surge Alerts",
17+
"surgeAlertImmediately": "Immediately"
1718
}
1819
}

src/views/SurgeOverview/SurgeAlertsTable/index.tsx

Lines changed: 23 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { useMemo, useCallback } from 'react';
2-
import { isNotDefined } from '@togglecorp/fujs';
2+
import { isDefined, isNotDefined } from '@togglecorp/fujs';
33

44
import Table from '#components/Table';
55
import Link from '#components/Link';
@@ -24,11 +24,6 @@ type SurgeAlertListItem = NonNullable<GetSurgeAlertResponse['results']>[number];
2424

2525
const surgeAlertKeySelector = (item: SurgeAlertListItem) => item.id;
2626

27-
// FIXME: use a separate utility
28-
const aMonthAgo = new Date();
29-
aMonthAgo.setMonth(aMonthAgo.getMonth() - 1);
30-
aMonthAgo.setHours(0, 0, 0, 0);
31-
3227
const today = new Date();
3328
const todayTimestamp = today.getTime();
3429

@@ -121,7 +116,27 @@ function SurgeAlertsTable() {
121116
createDateColumn<SurgeAlertListItem, number>(
122117
'start',
123118
strings.surgeAlertsTableStartDate,
124-
(surgeAlert) => surgeAlert.start,
119+
(surgeAlert) => {
120+
const startDate = isDefined(surgeAlert.start)
121+
? new Date(surgeAlert.start) : undefined;
122+
const endDate = isDefined(surgeAlert.end) ? new Date(surgeAlert.end) : undefined;
123+
124+
const closed = isDefined(surgeAlert.end)
125+
? new Date(surgeAlert.end).getTime() < todayTimestamp : undefined;
126+
127+
if (isDefined(endDate) && closed) {
128+
return endDate.toLocaleString();
129+
}
130+
131+
if (isDefined(startDate)) {
132+
const dateStarted = startDate.getTime() < todayTimestamp
133+
? strings.surgeAlertImmediately
134+
: startDate.toLocaleString();
135+
136+
return dateStarted;
137+
}
138+
return undefined;
139+
},
125140
),
126141
createStringColumn<SurgeAlertListItem, number>(
127142
'name',
@@ -162,6 +177,7 @@ function SurgeAlertsTable() {
162177
),
163178
]), [
164179
getStatus,
180+
strings.surgeAlertImmediately,
165181
strings.surgeAlertsTableAlertDate,
166182
strings.surgeAlertsTableDuration,
167183
strings.surgeAlertsTableStartDate,

0 commit comments

Comments
 (0)