Is a good/bad practice to use LazyQuery with async/await

[DirKraft91]

Hello guys, I have this part of code

const handleClick = useCallback(
    (event: React.MouseEvent) => {
      event.stopPropagation()

      if (hasInitMessages && messages) {
        updateOrCreateFolder(
          messages.map(uuid => ({ uuid })),
          folderName,
        )
      } else if (taskId && filterUuid && templateUuid) {
        query({
          taskId,
          templateUuid,
          filterUuid,
        })
      }
    },
    [filterUuid, folderName, hasInitMessages, messages, query, taskId, templateUuid, updateOrCreateFolder],
  )

  useEffect(() => {
    if (data) {
      updateOrCreateFolder(
        data.map(uuid => ({ uuid })),
        folderName,
      )
    }
    // cycle dependency.
    // eslint-disable-next-line react-hooks/exhaustive-deps
  }, [data, folderName])

I would like to use await query to avoid using useEffect with cycle dependency

1. Is in this case isLoading will work correctly?
2. Is it bad practice?

[phryneas]

Is in this case isLoading will work correctly?

Have you tried it?

Is it bad practice?

It's definitely better than using a mutation in a useEffect.