Skip to content

Commit 3fa9adb

Browse files
committed
fix: normalize default feedback date range
The initial feedback range should match setDateRange's startOfDay/endOfDay normalization. Otherwise the first "Last 30 days" window can exclude part of the boundary days until the picker changes. Keep the serialized useState initializer inline because this is a small JSON-safe default shape, not a reusable behavior. Nuxt useState docs: https://nuxt.com/docs/api/composables/use-state
1 parent 47e0744 commit 3fa9adb

1 file changed

Lines changed: 7 additions & 9 deletions

File tree

app/composables/useDateRange.ts

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -10,16 +10,14 @@ interface SerializedDateRange {
1010
end: string
1111
}
1212

13-
function createDefaultDateRange(): SerializedDateRange {
14-
const end = new Date()
15-
return {
16-
start: subDays(end, 30).toISOString(),
17-
end: end.toISOString()
18-
}
19-
}
20-
2113
export function useDateRange() {
22-
const serializedDateRange = useState<SerializedDateRange>('feedback-date-range', createDefaultDateRange)
14+
const serializedDateRange = useState<SerializedDateRange>('feedback-date-range', () => {
15+
const end = endOfDay(new Date())
16+
return {
17+
start: startOfDay(subDays(end, 30)).toISOString(),
18+
end: end.toISOString()
19+
}
20+
})
2321

2422
const dateRange = computed<DateRange>(() => ({
2523
start: new Date(serializedDateRange.value.start),

0 commit comments

Comments
 (0)