Skip to content

Commit

Permalink
Merge pull request #154 from concord-consortium/187852896-disable-dis…
Browse files Browse the repository at this point in the history
…able-inputs

fix: Disable disable controls for now [PT-187852896]
  • Loading branch information
dougmartin authored Jun 29, 2024
2 parents 4f48bc5 + b09ed84 commit 09b8bcc
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions src/shared/components/data-table-field.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -370,7 +370,9 @@ export const DataTableField: React.FC<FieldProps> = props => {
return;
}

setInputDisabled?.(true);
// TODO: RE-ENABLE
// DISABLED FOR NOW AS THIS BREAKS DATA SAVING
// setInputDisabled?.(true);

const timeSeriesMetadata = getTimeSeriesMetadata(timeSeriesCapabilities);

Expand Down Expand Up @@ -409,7 +411,9 @@ export const DataTableField: React.FC<FieldProps> = props => {
stopTimeSeriesFnRef.current = undefined;
timeSeriesRecordingRowRef.current = undefined;
saveData(finalData);
setInputDisabled?.(false);
// TODO: RE-ENABLE
// DISABLED FOR NOW AS THIS BREAKS DATA SAVING
// setInputDisabled?.(false);
}
};

Expand Down

0 comments on commit 09b8bcc

Please sign in to comment.