nit: use new util for clarity
This commit is contained in:
@@ -3,6 +3,7 @@ import {View} from 'react-native'
|
|||||||
import {msg, Trans} from '@lingui/macro'
|
import {msg, Trans} from '@lingui/macro'
|
||||||
import {useLingui} from '@lingui/react'
|
import {useLingui} from '@lingui/react'
|
||||||
|
|
||||||
|
import {useCallOnce} from '#/lib/once'
|
||||||
import {EmptyState} from '#/view/com/util/EmptyState'
|
import {EmptyState} from '#/view/com/util/EmptyState'
|
||||||
import {atoms as a, useTheme, web} from '#/alf'
|
import {atoms as a, useTheme, web} from '#/alf'
|
||||||
import {Button, ButtonText} from '#/components/Button'
|
import {Button, ButtonText} from '#/components/Button'
|
||||||
@@ -38,13 +39,16 @@ export function DraftsListDialog({
|
|||||||
// Fire draft:listOpen metric when dialog opens and data is loaded
|
// Fire draft:listOpen metric when dialog opens and data is loaded
|
||||||
const draftCount = drafts.length
|
const draftCount = drafts.length
|
||||||
const isDataReady = !isLoading && data !== undefined
|
const isDataReady = !isLoading && data !== undefined
|
||||||
|
const onDraftListOpen = useCallOnce()
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
if (isDataReady) {
|
if (isDataReady) {
|
||||||
ax.metric('draft:listOpen', {
|
onDraftListOpen(() => {
|
||||||
draftCount,
|
ax.metric('draft:listOpen', {
|
||||||
|
draftCount,
|
||||||
|
})
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
}, [isDataReady, draftCount, ax])
|
}, [onDraftListOpen, isDataReady, draftCount, ax])
|
||||||
|
|
||||||
const handleSelectDraft = useCallback(
|
const handleSelectDraft = useCallback(
|
||||||
(summary: DraftSummary) => {
|
(summary: DraftSummary) => {
|
||||||
|
|||||||
Reference in New Issue
Block a user