farmcontrol/farmcontrol-ui
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Refactor object data merging in NewObjectForm to use lodash's merge for improved state management

Browse Source
This commit is contained in:
Tom Butcher 2025-09-05 23:15:42 +01:00
parent c5355c8cf2
commit 204a881dfa
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/components/Dashboard/common/NewObjectForm.jsx
View File

@ -2,6 +2,7 @@ import { useState, useEffect, useContext } from 'react'
import { Form, message } from 'antd' import { Form, message } from 'antd'
import { ApiServerContext } from '../context/ApiServerContext' import { ApiServerContext } from '../context/ApiServerContext'
import PropTypes from 'prop-types' import PropTypes from 'prop-types'
import merge from 'lodash/merge'
/** /**
* NewObjectForm is a reusable form component for creating new objects. * NewObjectForm is a reusable form component for creating new objects.
@ -66,7 +67,9 @@ const NewObjectForm = ({ type, style, defaultValues = {}, children }) => {
layout='vertical' layout='vertical'
style={style} style={style}
onValuesChange={(values) => { onValuesChange={(values) => {
setObjectData((prev) => ({ ...prev, ...values })) setObjectData((prev) => {
return merge({}, prev, values)
})
}} }}
> >
{contextHolder} {contextHolder}