RealmInputScreen [nfc]: Have tryRealm take an arg for what realm to…

… try
chrisbobbe · Mar 10, 2022 · 87d4cf4 · 87d4cf4
1 parent 50b5a5a
commit 87d4cf4
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/src/common/SmartUrlInput.js b/src/common/SmartUrlInput.js
@@ -29,7 +29,7 @@ type Props = $ReadOnly<{|
   style?: ViewStyleProp,
   onChangeText: (value: string) => void,
   value: string,
-  onSubmitEditing: () => Promise<void>,
+  onSubmitEditing: () => void,
   enablesReturnKeyAutomatically: boolean,
 |}>;
 

diff --git a/src/start/RealmInputScreen.js b/src/start/RealmInputScreen.js
@@ -32,8 +32,8 @@ export default function RealmInputScreen(props: Props): Node {
 
   const [error, setError] = useState<string | null>(null);
 
-  const tryRealm = useCallback(async () => {
-    const parsedRealm = tryParseUrl(realmInputValue);
+  const tryRealm = useCallback(async (unparsedUrl: string) => {
+    const parsedRealm = tryParseUrl(unparsedUrl);
     if (!parsedRealm) {
       setError('Please enter a valid URL');
       return;
@@ -57,7 +57,11 @@ export default function RealmInputScreen(props: Props): Node {
     } finally {
       setProgress(false);
     }
-  }, [realmInputValue]);
+  }, []);
+
+  const handleInputSubmit = useCallback(() => {
+    tryRealm(realmInputValue);
+  }, [tryRealm, realmInputValue]);
 
   const styles = {
     input: { marginTop: 16, marginBottom: 8 },
@@ -80,7 +84,7 @@ export default function RealmInputScreen(props: Props): Node {
         navigation={navigation}
         onChangeText={setRealmInputValue}
         value={realmInputValue}
-        onSubmitEditing={tryRealm}
+        onSubmitEditing={handleInputSubmit}
         enablesReturnKeyAutomatically
       />
       {error !== null ? (
@@ -92,7 +96,7 @@ export default function RealmInputScreen(props: Props): Node {
         style={styles.button}
         text="Enter"
         progress={progress}
-        onPress={tryRealm}
+        onPress={handleInputSubmit}
         disabled={tryParseUrl(realmInputValue) === undefined}
       />
     </Screen>