Skip to content

Commit b311ea5

Browse files
Merge pull request #3754 from RedisInsight/fe/bugfix/RI-6044-timestamp-formatter-not-applied-when-no-settings
RI-6044 format datetime default if no settings provided
2 parents 5d8fc9b + 235ebea commit b311ea5

File tree

2 files changed

+13
-4
lines changed

2 files changed

+13
-4
lines changed

redisinsight/ui/src/pages/browser/modules/key-details/components/stream-details/stream-data-view/StreamDataViewWrapper.tsx

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ import {
1616
import { streamDataSelector, deleteStreamEntry } from 'uiSrc/slices/browser/stream'
1717
import { ITableColumn } from 'uiSrc/components/virtual-table/interfaces'
1818
import PopoverDelete from 'uiSrc/pages/browser/components/popover-delete/PopoverDelete'
19-
import { KeyTypes, TableCellTextAlignment, TEXT_FAILED_CONVENT_FORMATTER } from 'uiSrc/constants'
19+
import { DATETIME_FORMATTER_DEFAULT, KeyTypes, TableCellTextAlignment, TEXT_FAILED_CONVENT_FORMATTER, TimezoneOption } from 'uiSrc/constants'
2020
import { getBasedOnViewTypeEvent, sendEventTelemetry, TelemetryEvent } from 'uiSrc/telemetry'
2121
import { connectedInstanceSelector } from 'uiSrc/slices/instances/instances'
2222
import { keysSelector, selectedKeySelector, updateSelectedKeyRefreshTime } from 'uiSrc/slices/browser/keys'
@@ -249,7 +249,11 @@ const StreamDataViewWrapper = (props: Props) => {
249249
render: function Id({ id }: StreamEntryDto) {
250250
const idStr = bufferToString(id, viewFormat)
251251
const timestamp = idStr.split('-')?.[0]
252-
const formattedTimestamp = timestamp.length > MAX_FORMAT_LENGTH_STREAM_TIMESTAMP ? '-' : formatTimestamp(timestamp, config?.dateFormat, config?.timezone)
252+
const formattedTimestamp = timestamp.length > MAX_FORMAT_LENGTH_STREAM_TIMESTAMP ? '-' : formatTimestamp(
253+
timestamp,
254+
config?.dateFormat || DATETIME_FORMATTER_DEFAULT,
255+
config?.timezone || TimezoneOption.Local
256+
)
253257

254258
return (
255259
<div>

redisinsight/ui/src/utils/formatters/valueFormatters.tsx

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import JSONBigInt from 'json-bigint'
99
import { store } from 'uiSrc/slices/store'
1010

1111
import JSONViewer from 'uiSrc/components/json-viewer/JSONViewer'
12-
import { KeyValueFormat } from 'uiSrc/constants'
12+
import { DATETIME_FORMATTER_DEFAULT, KeyValueFormat, TimezoneOption } from 'uiSrc/constants'
1313
import { RedisResponseBuffer } from 'uiSrc/slices/interfaces'
1414
import {
1515
anyToBuffer,
@@ -177,7 +177,12 @@ const formattingBuffer = (
177177
// if seconds - add milliseconds (since JS Date works only with milliseconds)
178178
const timestamp = convertTimestampToMilliseconds(value)
179179
const config = get(store.getState(), 'user.settings.config', null)
180-
return { value: formatTimestamp(timestamp, config?.dateFormat, config?.timezone), isValid: true }
180+
return { value: formatTimestamp(
181+
timestamp,
182+
config?.dateFormat || DATETIME_FORMATTER_DEFAULT,
183+
config?.timezone || TimezoneOption.Local,
184+
),
185+
isValid: true }
181186
}
182187
} catch (e) {
183188
// if error return default

0 commit comments

Comments
 (0)