Skip to content

Commit

Permalink
Merge pull request #274 from bcgov/hotfix/request-get-put
Browse files Browse the repository at this point in the history
Avoid making an edit call if it's on first load.
  • Loading branch information
BrandonSharratt authored Jan 27, 2020
2 parents 3822bc5 + aeb8faf commit 3c7c38a
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion frontend/src/modules/requests/components/request/details.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ function RequestDetails({
isLoading,
onSave,
}) {
const editingRef = React.useRef(isEditing);
const formRef = React.useRef({});
const files = get(data, 'files', []);
const supportingFiles = get(data, 'supportingFiles', []);
Expand All @@ -46,9 +47,11 @@ function RequestDetails({
}, [data]);

React.useEffect(() => {
if (!isEditing) {
if (!isEditing && editingRef.current === true) {
onSave(id, formRef.current);
}

editingRef.current = isEditing;
}, [isEditing]);

if (isLoading && !data._id) {
Expand Down

0 comments on commit 3c7c38a

Please sign in to comment.