Refactor duplicate installation handling in AppUpdateProvider
All checks were successful
farmcontrol/farmcontrol-ui/pipeline/head This commit looks good

- Updated state management for duplicate installations by setting default values to null, simplifying the logic for handling duplicates.
- Changed the initial state of duplicatePromptOpen to false, improving the user experience by preventing unnecessary prompts on load.
This commit is contained in:
Tom Butcher 2026-08-09 11:19:55 +01:00
parent b254c0c94d
commit 235b038ecc

View File

@ -145,11 +145,8 @@ export const AppUpdateProvider = ({ children }) => {
const [installingUpdate, setInstallingUpdate] = useState(null)
const [updateProgress, setUpdateProgress] = useState(null)
const [completedUpdate, setCompletedUpdate] = useState(null)
const [duplicateInstall, setDuplicateInstall] = useState({
duplicatePath: 'C:\\Program Files\\Farm Control',
originalPath: 'C:\\Program Files\\Farm Control'
})
const [duplicatePromptOpen, setDuplicatePromptOpen] = useState(true)
const [duplicateInstall, setDuplicateInstall] = useState(null)
const [duplicatePromptOpen, setDuplicatePromptOpen] = useState(false)
const [removingDuplicate, setRemovingDuplicate] = useState(false)
const [duplicateRemovalResult, setDuplicateRemovalResult] = useState(null)
const runningCheckRef = useRef(null)