From 92caf9e164e86d238bd4cb881fd1cfbb664021b1 Mon Sep 17 00:00:00 2001 From: psteinroe Date: Tue, 8 Aug 2023 15:36:44 +0200 Subject: [PATCH] fix: remove unnecessary useMemo --- .changeset/nice-balloons-suffer.md | 5 +++++ .../query/use-infinite-offset-pagination-query.ts | 10 +++------- .../src/query/use-offset-infinite-query.ts | 2 +- .../src/query/use-offset-infinite-scroll-query.ts | 13 ++++--------- 4 files changed, 13 insertions(+), 17 deletions(-) create mode 100644 .changeset/nice-balloons-suffer.md diff --git a/.changeset/nice-balloons-suffer.md b/.changeset/nice-balloons-suffer.md new file mode 100644 index 00000000..2d10c27a --- /dev/null +++ b/.changeset/nice-balloons-suffer.md @@ -0,0 +1,5 @@ +--- +"@supabase-cache-helpers/postgrest-swr": patch +--- + +fix: remove unnecessary useMemo calls diff --git a/packages/postgrest-swr/src/query/use-infinite-offset-pagination-query.ts b/packages/postgrest-swr/src/query/use-infinite-offset-pagination-query.ts index 8732ebd5..d03d3f53 100644 --- a/packages/postgrest-swr/src/query/use-infinite-offset-pagination-query.ts +++ b/packages/postgrest-swr/src/query/use-infinite-offset-pagination-query.ts @@ -94,13 +94,9 @@ function useInfiniteOffsetPaginationQuery< const [currentPageIndex, setCurrentPageIndex] = useState(0); - const { data: parsedData, hasMore } = useMemo(() => { - return { - data: (data ?? []).map((p) => p.data), - hasMore: - Array.isArray(data) && data.length > 0 && data[data.length - 1].hasMore, - }; - }, [data]); + const parsedData = (data ?? []).map((p) => p.data); + const hasMore = + Array.isArray(data) && data.length > 0 && data[data.length - 1].hasMore; return { pages: parsedData, diff --git a/packages/postgrest-swr/src/query/use-offset-infinite-query.ts b/packages/postgrest-swr/src/query/use-offset-infinite-query.ts index 9d5dda5a..4c66d6e7 100644 --- a/packages/postgrest-swr/src/query/use-offset-infinite-query.ts +++ b/packages/postgrest-swr/src/query/use-offset-infinite-query.ts @@ -48,7 +48,7 @@ function useOffsetInfiniteQuery< Relationships > | null, config?: SWRInfiniteConfiguration & { pageSize?: number } -): UseInfiniteQueryReturn { +): UseOffsetInfiniteQueryReturn { return useSWRInfinite< Exclude['data'], null>, PostgrestError diff --git a/packages/postgrest-swr/src/query/use-offset-infinite-scroll-query.ts b/packages/postgrest-swr/src/query/use-offset-infinite-scroll-query.ts index 730ca9fe..d3c0793a 100644 --- a/packages/postgrest-swr/src/query/use-offset-infinite-scroll-query.ts +++ b/packages/postgrest-swr/src/query/use-offset-infinite-scroll-query.ts @@ -77,7 +77,7 @@ function useOffsetInfiniteScrollQuery< Relationships > | null, config?: SWRInfiniteConfiguration & { pageSize?: number } -): UseInfiniteScrollQueryReturn { +): UseOffsetInfiniteScrollQueryReturn { const { data, setSize, size, isValidating, ...rest } = useSWRInfinite< PostgrestHasMorePaginationResponse, PostgrestError @@ -106,16 +106,11 @@ function useOffsetInfiniteScrollQuery< } ); - const { data: flatData, hasMore } = useMemo(() => { - return { - data: (data ?? []).flatMap((p) => p.data), - hasMore: - Array.isArray(data) && data.length > 0 && data[data.length - 1].hasMore, - }; - }, [data]); + const hasMore = + Array.isArray(data) && data.length > 0 && data[data.length - 1].hasMore; return { - data: flatData, + data: (data ?? []).flatMap((p) => p.data), size, setSize, loadMore: hasMore ? () => setSize(size + 1) : null,