Skip to content

Commit 98d672d

Browse files
committed
PR Feedback; date fmt update and better comments
1 parent 08fd58a commit 98d672d

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

site/frontend/src/utils/formatting.ts

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,9 @@ export function formatSecondsAsDuration(time: number): string {
2121
}
2222

2323
export const DATE_FMT_KEY = "__rustc-perf-user-date-fmt-preference__";
24+
// Date formats taken from https://date-fns.org/v4.1.0/docs/format
2425
export const DATE_FMT_24HR = "yyyy-MM-dd HH:mm:ss";
25-
export const DATE_FMT_12HR = "yyyy-MM-dd pp";
26+
export const DATE_FMT_12HR = "yyyy-MM-dd hh:mm:ss a";
2627

2728
export function setDateFmt(dateFmt: string) {
2829
window.localStorage.setItem(DATE_FMT_KEY, dateFmt);
@@ -32,10 +33,11 @@ export function getDateFmt() {
3233
return window.localStorage.getItem(DATE_FMT_KEY) ?? DATE_FMT_24HR;
3334
}
3435

35-
// Takes a date like `2025-09-10T08:22:47.161348Z` -> `"2025-09-10 08:22:47"`
36-
export function formatISODate(dateString?: string, fmt?: string): string {
36+
// Takes a date like `2025-09-10T08:22:47.161348Z` and formats it according to
37+
// the user preference stored in local storage (either 12 hour or 24 hour format).
38+
export function formatISODate(dateString?: string): string {
3739
if (dateString) {
38-
const dateFmt = fmt ?? getDateFmt();
40+
const dateFmt = getDateFmt();
3941
return format(parseISO(dateString), dateFmt);
4042
}
4143
return "";

0 commit comments

Comments
 (0)