Fix "useENSName" Hook (#578)

* Fix "useENSName" Hook * Check for "stale" in "useENSName" Hook

Fix "useENSName" Hook (#578)
* Fix "useENSName" Hook * Check for "stale" in "useENSName" Hook
701cbde2 · Paul Razvan Berg · Noah Zinsmeister · e00e7af5 · 701cbde2
Commit 701cbde2 authored Dec 31, 2019 by Paul Razvan Berg Committed by Noah Zinsmeister Dec 31, 2019
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 5 deletions

index.js src/hooks/index.js +8 -5

No files found.
--- a/src/hooks/index.js
+++ b/src/hooks/index.js
@@ -146,8 +146,9 @@ export function useENSName(address) {
  useEffect(() => {
    if (isAddress(address)) {
      let stale = false
-      try {
-        library.lookupAddress(address).then(name => {
+      library
+        .lookupAddress(address)
+        .then(name => {
          if (!stale) {
            if (name) {
              setENSName(name)
@@ -156,9 +157,11 @@ export function useENSName(address) {
            }
          }
        })
-      } catch {
-        setENSName(null)
-      }
+        .catch(() => {
+          if (!stale) {
+            setENSName(null)
+          }
+        })

      return () => {
        stale = true