Refactor useEffectfulState to cater reset input UI rerendering (#1009)

* Refactor useEffectfulState to cater reset input UI rerendering * Add changeset * Update changeset
seek-oss · Sep 22, 2023 · b2e87be · b2e87be
1 parent 32f01f9
commit b2e87be
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/.changeset/slimy-guests-melt.md b/.changeset/slimy-guests-melt.md
@@ -0,0 +1,5 @@
+---
+'wingman-fe': patch
+---
+
+SalaryDetails: Update state fields on empty strings
diff --git a/fe/lib/components/SalaryDetails/SalaryDetails.tsx b/fe/lib/components/SalaryDetails/SalaryDetails.tsx
@@ -62,8 +62,9 @@ export interface SalaryDetailsProps {
 function useEffectfulState<T>(initialState: T) {
   const [state, setState] = useState(initialState);
 
+  // Make sure we rerender the input UI when the state value is reset
   useEffect(() => {
-    if (initialState) {
+    if (initialState || initialState === '') {
       setState(initialState);
     }
   }, [initialState]);