fix: hostd announce button states

.changeset/rotten-dots-reply.md

@@ -0,0 +1,5 @@
+---
+'hostd': minor
+---
+
+The announce button is now disabled unless the net address does not match the last announcement. Closes https://github.com/SiaFoundation/hostd/issues/248

apps/hostd/components/Config/AnnounceButton.tsx

@@ -1,12 +1,18 @@
 import {
   Button,
   Paragraph,
+  minutesInMilliseconds,
   triggerErrorToast,
   triggerSuccessToast,
 } from '@siafoundation/design-system'
 import { Bullhorn16 } from '@siafoundation/react-icons'
 import { useDialog } from '../../contexts/dialog'
-import { useSettingsAnnounce, useTxPoolFee } from '@siafoundation/react-hostd'
+import {
+  useSettings,
+  useSettingsAnnounce,
+  useStateHost,
+  useTxPoolFee,
+} from '@siafoundation/react-hostd'
 import { humanSiacoin } from '@siafoundation/units'
 import { useCallback } from 'react'
 import BigNumber from 'bignumber.js'
@@ -15,6 +21,17 @@ export function AnnounceButton() {
   const { openConfirmDialog } = useDialog()
   const txpoolFee = useTxPoolFee()
   const settingsAnnounce = useSettingsAnnounce()
+  const host = useStateHost()
+  const settings = useSettings({
+    config: {
+      swr: {
+        refreshInterval: minutesInMilliseconds(1),
+      },
+    },
+  })
+
+  const needsToAnnounce =
+    host.data.lastAnnouncement?.address !== settings.data?.netAddress
 
   const triggerConfirm = useCallback(
     () =>
@@ -54,7 +71,11 @@ export function AnnounceButton() {
   )
 
   return (
-    <Button tip="Announce host address" onClick={triggerConfirm}>
+    <Button
+      tip="Announce host address"
+      onClick={triggerConfirm}
+      disabled={!needsToAnnounce}
+    >
       <Bullhorn16 />
       Announce
     </Button>

apps/hostd/contexts/config/index.tsx

@@ -74,7 +74,6 @@ export function useConfigMain() {
 
   const onValid = useOnValid({
     resources,
-    dirtyFields: form.formState.dirtyFields,
     showAdvanced,
     revalidateAndResetForm,
   })

apps/hostd/contexts/config/useOnValid.tsx

@@ -1,26 +1,31 @@
 import {
   triggerSuccessToast,
   triggerErrorToast,
+  minutesInMilliseconds,
 } from '@siafoundation/design-system'
 import { useCallback } from 'react'
 import { SettingsData, initialValues } from './types'
 import { calculateMaxCollateral, transformUp } from './transform'
-import { UseFormReturn } from 'react-hook-form'
 import { Resources } from './resources'
-import { useSettingsUpdate } from '@siafoundation/react-hostd'
+import { useSettingsUpdate, useStateHost } from '@siafoundation/react-hostd'
 
 export function useOnValid({
   resources,
-  dirtyFields,
   showAdvanced,
   revalidateAndResetForm,
 }: {
-  dirtyFields: UseFormReturn<SettingsData>['formState']['dirtyFields']
   resources: Resources
   showAdvanced: boolean
   revalidateAndResetForm: () => Promise<void>
 }) {
   const settingsUpdate = useSettingsUpdate()
+  const host = useStateHost({
+    config: {
+      swr: {
+        refreshInterval: minutesInMilliseconds(1),
+      },
+    },
+  })
   const onValid = useCallback(
     async (values: typeof initialValues) => {
       if (!resources) {
@@ -46,7 +51,9 @@ export function useOnValid({
         if (response.error) {
           throw Error(response.error)
         }
-        if (dirtyFields.netAddress) {
+        const needsToAnnounce =
+          host.data.lastAnnouncement?.address !== values.netAddress
+        if (needsToAnnounce) {
           triggerSuccessToast(
             'Settings have been saved. Address has changed, make sure to re-announce the host.',
             {
@@ -62,13 +69,7 @@ export function useOnValid({
         console.log(e)
       }
     },
-    [
-      showAdvanced,
-      resources,
-      dirtyFields.netAddress,
-      settingsUpdate,
-      revalidateAndResetForm,
-    ]
+    [showAdvanced, resources, settingsUpdate, revalidateAndResetForm, host]
   )
   return onValid
 }

libs/react-hostd/src/api.ts

@@ -18,20 +18,28 @@ import {
   PublicKey,
   Currency,
   TransactionID,
+  ChainIndex,
 } from '@siafoundation/types'
 import useSWR from 'swr'
 import { Contract, ContractStatus, WalletTransaction } from './siaTypes'
 
 // state
 
 export type StateHost = {
+  name?: string
   publicKey: string
   walletAddress: string
   network: 'Mainnet' | 'Zen Testnet'
   version: string
   commit: string
   os: string
+  startTime: string
   buildTime: string
+  lastAnnouncement?: {
+    index: ChainIndex
+    publicKey: string
+    address: string
+  }
 }
 
 export const stateHostKey = '/state/host'