fix(selection-list): make showTooltip required

Expensify · Aug 30, 2023 · 0b2d4b4 · 0b2d4b4
1 parent 5c67b87
commit 0b2d4b4
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/src/components/SelectionList/BaseSelectionList.js b/src/components/SelectionList/BaseSelectionList.js
@@ -266,7 +266,6 @@ function BaseSelectionList({
                 isFocused={isFocused}
                 isDisabled={isDisabled}
                 onSelectRow={() => selectRow(item, index)}
-                showTooltip={showTooltip}
             />
         );
     };

diff --git a/src/components/SelectionList/UserListItem.js b/src/components/SelectionList/UserListItem.js
@@ -16,7 +16,7 @@ import themeColors from '../../styles/themes/default';
 import Tooltip from '../Tooltip';
 import UserDetailsTooltip from '../UserDetailsTooltip';
 
-function UserListItem({item, isFocused = false, showTooltip = false, onSelectRow, onDismissError = () => {}}) {
+function UserListItem({item, isFocused = false, showTooltip, onSelectRow, onDismissError = () => {}}) {
     const hasError = !_.isEmpty(item.errors);
 
     const avatar = (

diff --git a/src/components/SelectionList/selectionListPropTypes.js b/src/components/SelectionList/selectionListPropTypes.js
@@ -47,7 +47,7 @@ const userListItemPropTypes = {
     isFocused: PropTypes.bool,
 
     /** Whether this item should show Tooltip */
-    showTooltip: PropTypes.bool,
+    showTooltip: PropTypes.bool.isRequired,
 
     /** Callback to fire when the item is pressed */
     onSelectRow: PropTypes.func.isRequired,