Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ui: update all dates to use same format #83468

Merged
merged 1 commit into from
Jun 28, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ import {
ExecutionType,
ExecutionsColumn,
} from "../execTableCommon";
import { DATE_FORMAT } from "../../util";

interface ActiveStatementsTable {
data: ActiveStatement[];
Expand Down Expand Up @@ -73,8 +74,7 @@ export function makeActiveStatementsColumns(): ColumnDescriptor<ActiveStatement>
{
name: "startTime",
title: executionsTableTitles.startTime(execType),
cell: (item: ActiveStatement) =>
item.start.format("MMM D, YYYY [at] H:mm"),
cell: (item: ActiveStatement) => item.start.format(DATE_FORMAT),
sort: (item: ActiveStatement) => item.start.unix(),
},
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ import {
ExecutionType,
ExecutionsColumn,
} from "../execTableCommon";
import { DATE_FORMAT } from "../../util";

interface ActiveTransactionsTable {
data: ActiveTransaction[];
Expand Down Expand Up @@ -75,8 +76,7 @@ export function makeActiveTransactionsColumns(): ColumnDescriptor<ActiveTransact
{
name: "startTime",
title: executionsTableTitles.startTime(execType),
cell: (item: ActiveTransaction) =>
item.start.format("MMM D, YYYY [at] H:mm"),
cell: (item: ActiveTransaction) => item.start.format(DATE_FORMAT),
sort: (item: ActiveTransaction) => item.start.unix(),
},
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@ import {
} from "src/transactionsPage/transactionsPageClasses";
import { Moment } from "moment";
import { Caution } from "@cockroachlabs/icons";
import { DATE_FORMAT } from "src/util/format";

const cx = classNames.bind(styles);
const sortableTableCx = classNames.bind(sortableTableStyles);
Expand Down Expand Up @@ -379,7 +380,7 @@ export class DatabaseDetailsPage extends React.Component<
cell: table =>
!table.details.statsLastUpdated
? "No table statistics found"
: table.details.statsLastUpdated.format("MMM DD, YYYY [at] H:mm"),
: table.details.statsLastUpdated.format(DATE_FORMAT),
sort: table => table.details.statsLastUpdated,
className: cx("database-table__col--table-stats"),
name: "tableStatsUpdated",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@ import classnames from "classnames/bind";
import booleanSettingStyles from "../settings/booleanSetting.module.scss";
import { CircleFilled } from "../icon";
import { performanceTuningRecipes } from "src/util/docs";
import { DATE_FORMAT_24_UTC } from "src/util/format";
const cx = classNames.bind(styles);
const booleanSettingCx = classnames.bind(booleanSettingStyles);

Expand Down Expand Up @@ -241,9 +242,7 @@ export class DatabaseTablePage extends React.Component<
if (lastReset.isSame(this.minDate)) {
return "Last reset: Never";
} else {
return (
"Last reset: " + lastReset.format("MMM DD, YYYY [at] H:mm [(UTC)]")
);
return "Last reset: " + lastReset.format(DATE_FORMAT_24_UTC);
}
}

Expand Down
3 changes: 2 additions & 1 deletion pkg/ui/workspaces/cluster-ui/src/graphs/utils/domain.ts
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ import {
BytesFitScale,
ComputeByteScale,
ComputeDurationScale,
DATE_WITH_SECONDS_FORMAT_24_UTC,
DurationFitScale,
} from "src/util/format";

Expand Down Expand Up @@ -264,7 +265,7 @@ const timeIncrements: number[] = timeIncrementDurations.map(inc =>
);

export function formatTimeStamp(timeMillis: number): string {
return moment.utc(timeMillis).format("HH:mm:ss on MMM Do, YYYY");
return moment.utc(timeMillis).format(DATE_WITH_SECONDS_FORMAT_24_UTC);
}

function ComputeTimeAxisDomain(extent: Extent): AxisDomain {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ import { SummaryCard } from "../summaryCard";
import moment, { Moment } from "moment";
import { Heading } from "@cockroachlabs/ui-components";
import { Anchor } from "../anchor";
import { performanceTuningRecipes } from "../util";
import { DATE_FORMAT_24_UTC, performanceTuningRecipes } from "../util";

const cx = classNames.bind(styles);

Expand Down Expand Up @@ -127,7 +127,7 @@ export class IndexDetailsPage extends React.Component<
if (timestamp.isSame(minDate)) {
return "Never";
} else {
return timestamp.format("MMM DD, YYYY [at] H:mm [(UTC)]");
return timestamp.format(DATE_FORMAT_24_UTC);
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ import { StatusIcon } from "src/activeExecutions/statusIcon";

import styles from "./statementDetails.module.scss";
import { SqlBox } from "src/sql/box";
import { DATE_FORMAT_24_UTC } from "../util";
const cx = classNames.bind(styles);
const summaryCardStylesCx = classNames.bind(summaryCardStyles);

Expand Down Expand Up @@ -96,9 +97,7 @@ export const ActiveStatementDetails: React.FC<ActiveStatementDetailsProps> = ({
<Col>
<div className={summaryCardStylesCx("summary--card__item")}>
<Text>Start Time (UTC)</Text>
<Text>
{statement.start.format("MMM D, YYYY [at] H:mm (UTC)")}
</Text>
<Text>{statement.start.format(DATE_FORMAT_24_UTC)}</Text>
</div>
<div className={summaryCardStylesCx("summary--card__item")}>
<Text>Elapsed Time</Text>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ import {
import { EmptyTable } from "src/empty";
import styles from "./diagnosticsView.module.scss";
import { getBasePath } from "../../api";
import { DATE_FORMAT_24_UTC } from "../../util";

type IStatementDiagnosticsReport =
cockroach.server.serverpb.IStatementDiagnosticsReport;
Expand Down Expand Up @@ -127,7 +128,7 @@ export class DiagnosticsView extends React.Component<
defaultSortOrder: "descend",
render: (_text, record) => {
const timestamp = record.requested_at.seconds.toNumber() * 1000;
return moment.utc(timestamp).format("LL[ at ]H:mm");
return moment.utc(timestamp).format(DATE_FORMAT_24_UTC);
},
},
{
Expand Down
3 changes: 2 additions & 1 deletion pkg/ui/workspaces/cluster-ui/src/util/format.ts
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,8 @@ export const DATE_FORMAT = "MMM DD, YYYY [at] H:mm";
/**
* Alternate 24 hour UTC format
*/
export const DATE_FORMAT_24_UTC = "MMM DD, YYYY [at] HH:mm UTC";
export const DATE_FORMAT_24_UTC = "MMM DD, YYYY [at] H:mm UTC";
export const DATE_WITH_SECONDS_FORMAT_24_UTC = "MMM DD, YYYY [at] H:mm:ss UTC";

export function RenderCount(yesCount: Long, totalCount: Long): string {
if (longToInt(yesCount) == 0) {
Expand Down
3 changes: 2 additions & 1 deletion pkg/ui/workspaces/db-console/src/util/format.ts
Original file line number Diff line number Diff line change
Expand Up @@ -156,4 +156,5 @@ export const DATE_FORMAT = "MMM DD, YYYY [at] H:mm";
/**
* Alternate 24 hour UTC format
*/
export const DATE_FORMAT_24_UTC = "MMM DD, YYYY [at] HH:mm UTC";
export const DATE_FORMAT_24_UTC = "MMM DD, YYYY [at] H:mm UTC";
export const DATE_WITH_SECONDS_FORMAT_24_UTC = "MMM DD, YYYY [at] H:mm:ss UTC";
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ import { getDisplayName } from "src/redux/nodes";
import { Loading, SortedTable, util } from "@cockroachlabs/cluster-ui";
import { getMatchParamByName } from "src/util/query";
import "./logs.styl";
import { DATE_WITH_SECONDS_FORMAT_24_UTC } from "src/util/format";

type LogEntries = protos.cockroach.util.log.IEntry;

Expand Down Expand Up @@ -55,7 +56,9 @@ export class Logs extends React.Component<LogProps & RouteComponentProps, {}> {
title: "Time",
name: "time",
cell: (logEntry: LogEntries) =>
util.LongToMoment(logEntry.time).format("YYYY-MM-DD HH:mm:ss"),
util
.LongToMoment(logEntry.time)
.format(DATE_WITH_SECONDS_FORMAT_24_UTC),
},
{
title: "Severity",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ import {
SortSetting,
util,
} from "@cockroachlabs/cluster-ui";
import { Percentage } from "src/util/format";
import { DATE_FORMAT_24_UTC, Percentage } from "src/util/format";
import { FixLong } from "src/util/fixLong";
import { getNodeLocalityTiers } from "src/util/localities";
import { LocalityTier } from "src/redux/localities";
Expand Down Expand Up @@ -440,7 +440,7 @@ class DecommissionedNodeList extends React.Component<DecommissionedNodeListProps
key: "decommissionedSince",
title: "decommissioned on",
render: (_text: string, record: DecommissionedNodeStatusRow) =>
record.decommissionedDate.format("LL[ at ]H:mm UTC"),
record.decommissionedDate.format(DATE_FORMAT_24_UTC),
},
{
key: "status",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ import {
util,
} from "@cockroachlabs/cluster-ui";
import { createSelector } from "reselect";
import { DATE_FORMAT_24_UTC } from "src/util/format";

const decommissionedNodesSortSetting = new LocalSetting<
AdminUIState,
Expand Down Expand Up @@ -88,7 +89,7 @@ export class DecommissionedNodeHistory extends React.Component<DecommissionedNod
title: "Decommissioned On",
sorter: sortByDecommissioningDate,
render: (_text, record) => {
return record.decommissionedDate.format("LL[ at ]H:mm UTC");
return record.decommissionedDate.format(DATE_FORMAT_24_UTC);
},
},
];
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@ import {
} from "@cockroachlabs/cluster-ui";
import { cancelStatementDiagnosticsReportAction } from "src/redux/statements";
import { trackCancelDiagnosticsBundleAction } from "src/redux/analyticsActions";
import { DATE_FORMAT_24_UTC } from "src/util/format";

type StatementDiagnosticsHistoryViewProps = MapStateToProps &
MapDispatchToProps;
Expand Down Expand Up @@ -98,7 +99,7 @@ class StatementDiagnosticsHistoryView extends React.Component<
cell: record =>
moment
.utc(record.requested_at.seconds.toNumber() * 1000)
.format("LL[ at ]H:mm"),
.format(DATE_FORMAT_24_UTC),
sort: record => moment(record.requested_at.seconds.toNumber() * 1000),
},
{
Expand Down