hotel-manage-stack/cashup
1
0
Fork 0
Code Issues 2 Pull requests Projects Releases Packages Wiki Activity Actions

Fix history links opening today's cash up instead of the target date

Browse source 
Daily Cash Up initialised its date state from today() only, ignoring
the ?date= query param History links navigate with. Read it from the
URL on mount instead.
This commit is contained in:
jtricerolph 2026-08-03 08:56:35 +00:00
parent 2e9b4a2ca3
commit 5c5d051c35
1 changed files with 3 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

4
frontend/src/pages/DailyCashUp.tsx
View file

@ -1,4 +1,5 @@
import { useState, useEffect, useRef, useCallback } from 'react' import { useState, useEffect, useRef, useCallback } from 'react'
import { useSearchParams } from 'react-router-dom'
import { RefreshCw, Save, CheckCircle, Loader, Camera, FileText, X } from 'lucide-react' import { RefreshCw, Save, CheckCircle, Loader, Camera, FileText, X } from 'lucide-react'
import { api, uploadAttachment } from '../api' import { api, uploadAttachment } from '../api'
import { PageHeader, Card, Btn, StatusBadge } from '../components/Layout' import { PageHeader, Card, Btn, StatusBadge } from '../components/Layout'
@ -37,7 +38,8 @@ type PageState = 'checking' | 'empty' | 'editing' | 'locked'
export function DailyCashUp({ user }: Props) { export function DailyCashUp({ user }: Props) {
const canFinalise = can(user, 'finalise') const canFinalise = can(user, 'finalise')
const [date, setDate] = useState(today()) const [searchParams] = useSearchParams()
const [date, setDate] = useState(searchParams.get('date') || today())
const [pageState, setPageState] = useState<PageState>('checking') const [pageState, setPageState] = useState<PageState>('checking')
const [cashUp, setCashUp] = useState<CashUp | null>(null) const [cashUp, setCashUp] = useState<CashUp | null>(null)
const [takings, setTakings] = useState<Denomination[]>(initDenominations('takings')) const [takings, setTakings] = useState<Denomination[]>(initDenominations('takings'))