Merge pull request #911 from ensdomains/fet-1689-alternate

src/hooks/pages/profile/useRenew/useRenew.ts

@@ -1,50 +1,123 @@
 import { useConnectModal } from '@rainbow-me/rainbowkit'
 import { useSearchParams } from 'next/navigation'
-import { useEffect, useRef } from 'react'
+import { useEffect, useState } from 'react'
+import { match } from 'ts-pattern'
 import { useAccount } from 'wagmi'
 
 import { useAbilities } from '@app/hooks/abilities/useAbilities'
-import { useNameDetails } from '@app/hooks/useNameDetails'
+import { useBasicName } from '@app/hooks/useBasicName'
+import { useRouterWithHistory } from '@app/hooks/useRouterWithHistory'
 import { useTransactionFlow } from '@app/transaction-flow/TransactionFlowProvider'
+import { RegistrationStatus } from '@app/utils/registrationStatus'
 import { parseNumericString } from '@app/utils/string'
 
+type RenewStatus = 'connect-user' | 'display-extend-names' | 'idle'
+
+export const calculateRenewState = ({
+  registrationStatus,
+  isRegistrationStatusLoading,
+  renewSeconds,
+  openConnectModal,
+  connectModalOpen,
+  accountStatus,
+  isAbilitiesLoading,
+  isRouterReady,
+  name,
+  openedConnectModal,
+}: {
+  registrationStatus?: RegistrationStatus
+  isRegistrationStatusLoading: boolean
+  renewSeconds: number | null
+  connectModalOpen: boolean
+  openConnectModal: ReturnType<typeof useConnectModal>['openConnectModal']
+  accountStatus: ReturnType<typeof useAccount>['status']
+  isAbilitiesLoading: boolean
+  isRouterReady: boolean
+  name?: string
+  openedConnectModal: boolean
+}): RenewStatus => {
+  const isNameRegisteredOrGracePeriod =
+    registrationStatus === 'registered' || registrationStatus === 'gracePeriod'
+  const isRenewActive =
+    isRouterReady &&
+    !isRegistrationStatusLoading &&
+    !!name &&
+    isNameRegisteredOrGracePeriod &&
+    !!renewSeconds &&
+    !connectModalOpen
+
+  if (
+    isRenewActive &&
+    accountStatus === 'disconnected' &&
+    !!openConnectModal &&
+    !openedConnectModal
+  )
+    return 'connect-user'
+  if (isRenewActive && accountStatus === 'connected' && !isAbilitiesLoading)
+    return 'display-extend-names'
+  return 'idle'
+}
+
+export const removeRenewParam = ({
+  query,
+}: {
+  query: ReturnType<typeof useRouterWithHistory>['query']
+}): string => {
+  const searchParams = new URLSearchParams(query as any)
+  // remove the name param in case the page is a redirect from /profile page
+  searchParams.delete('name')
+  searchParams.delete('renew')
+  const newParams = searchParams.toString()
+  return newParams ? `?${newParams}` : ''
+}
+
 export function useRenew(name: string) {
-  const { registrationStatus, isLoading } = useNameDetails({ name })
+  const router = useRouterWithHistory()
+  const { registrationStatus, isLoading: isBasicNameLoading } = useBasicName({ name })
   const abilities = useAbilities({ name })
   const searchParams = useSearchParams()
-  const { isConnected, isDisconnected } = useAccount()
-  const { usePreparedDataInput } = useTransactionFlow()
+  const { status } = useAccount()
+
   const { openConnectModal, connectModalOpen } = useConnectModal()
+  const [openedConnectModal, setOpenedConnectModal] = useState(connectModalOpen)
+
+  const { usePreparedDataInput } = useTransactionFlow()
   const showExtendNamesInput = usePreparedDataInput('ExtendNames')
 
-  const { data: { canSelfExtend } = {} } = abilities
-  const isAvailableName = registrationStatus === 'available'
-  const renewSeconds = parseNumericString(searchParams.get('renew'))
+  const { data: { canSelfExtend } = {}, isLoading: isAbilitiesLoading } = abilities
 
-  const prevIsConnected = useRef(isConnected)
+  const renewSeconds = parseNumericString(searchParams.get('renew'))
 
-  const isRenewActive =
-    (!isConnected || !connectModalOpen) && !!renewSeconds && !isLoading && !isAvailableName
+  const renewState = calculateRenewState({
+    registrationStatus,
+    isRegistrationStatusLoading: isBasicNameLoading,
+    renewSeconds,
+    connectModalOpen,
+    accountStatus: status,
+    isAbilitiesLoading,
+    name,
+    isRouterReady: router.isReady,
+    openConnectModal,
+    openedConnectModal,
+  })
 
-  // http://localhost:3000/anyname.eth?renew=63072000
   useEffect(() => {
-    if (!isRenewActive) return
-
-    if (isDisconnected && prevIsConnected.current) {
-      openConnectModal?.()
-      return
-    }
-
-    if (isConnected) {
-      showExtendNamesInput(`extend-names-${name}`, {
-        names: [name],
-        isSelf: canSelfExtend,
-        seconds: renewSeconds,
+    match(renewState)
+      .with('connect-user', () => {
+        openConnectModal?.()
+        setOpenedConnectModal(!!openConnectModal)
       })
-    }
-
-    prevIsConnected.current = !isConnected
-
+      .with('display-extend-names', () => {
+        showExtendNamesInput(`extend-names-${name}`, {
+          names: [name],
+          isSelf: canSelfExtend,
+          seconds: renewSeconds!,
+        })
+        const params = removeRenewParam({ query: router.query })
+        router.replace(`/${name}${params}`)
+      })
+      .with('idle', () => {})
+      .exhaustive()
     // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [isRenewActive, isConnected, isDisconnected, name, canSelfExtend])
+  }, [renewState])
 }