track false fetching state when loading pages

HoudiniGraphql · Nov 10, 2022 · 298ce49 · 298ce49
1 parent cf4212a
commit 298ce49
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/packages/houdini-svelte/src/runtime/stores/pagination/cursor.ts b/packages/houdini-svelte/src/runtime/stores/pagination/cursor.ts
@@ -99,6 +99,9 @@ export function cursorHandlers<_Data extends GraphQLObject, _Input>({
 			variables: loadVariables,
 			applyUpdates: true,
 		})
+
+		// we're not loading any more
+		setFetching(false)
 	}
 
 	return {
@@ -222,6 +225,9 @@ export function cursorHandlers<_Data extends GraphQLObject, _Input>({
 			// keep the page info store up to date
 			pageInfo.set(extractPageInfo(result.data, artifact.refetch!.path))
 
+			// we're not loading any more
+			setFetching(false)
+
 			return {
 				data: result.data,
 				variables: queryVariables as _Input,

diff --git a/packages/houdini-svelte/src/runtime/stores/pagination/offset.ts b/packages/houdini-svelte/src/runtime/stores/pagination/offset.ts
@@ -93,6 +93,9 @@ export function offsetHandlers<_Data extends GraphQLObject, _Input>({
 			// add the page size to the offset so we load the next page next time
 			const pageSize = queryVariables.limit || artifact.refetch!.pageSize
 			currentOffset = offset + pageSize
+
+			// we're not loading any more
+			setFetching(false)
 		},
 		async fetch(
 			args?: QueryStoreFetchParams<_Data, _Input>
@@ -128,6 +131,9 @@ export function offsetHandlers<_Data extends GraphQLObject, _Input>({
 				variables: queryVariables as _Input,
 			})
 
+			// we're not loading any more
+			setFetching(false)
+
 			return {
 				data: result.data,
 				variables: queryVariables as _Input,