@ethlete/query 5.43.1 → 5.44.1

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # @ethlete/query
 
+## 5.44.1
+
+### Patch Changes
+
+- [`2c02096`](https://github.com/ethlete-io/ethdk/commit/2c020964d1b57ebaa483035be59ee84a15d1794e) Thanks [@TomTomB](https://github.com/TomTomB)! - Give each object in an array its own index when `writeArrayIndexes` is enabled. Before, every object item was serialized at index `0`.
+
+## 5.44.0
+
+### Minor Changes
+
+- [`8d2efef`](https://github.com/ethlete-io/ethdk/commit/8d2efef2a367db82d44977a3fdfe56396bb46515) Thanks [@TomTomB](https://github.com/TomTomB)! - Minor bump only
+
+## 5.43.2
+
+### Patch Changes
+
+- [`830b987`](https://github.com/ethlete-io/ethdk/commit/830b987eba43a51252c3c55eee5590ceef8bf1cb) Thanks [@TomTomB](https://github.com/TomTomB)! - Fix query v3 features not reacting to 204 responses due to null checks
+
 ## 5.43.1
 
 ### Patch Changes

package/fesm2022/ethlete-query.mjs

@@ -140,21 +140,24 @@ const buildQueryString = (params, config) => {
         }
         else if (Array.isArray(value)) {
             let currentFilteredIndex = 0;
+            let didAddAnyValue = false;
             for (const arrayValue of value) {
                 const nestedKey = writeArrayIndexes ? `${key}[${currentFilteredIndex}]` : `${key}[]`;
                 const didAddValue = processValue(nestedKey, arrayValue);
                 if (didAddValue) {
                     currentFilteredIndex++;
+                    didAddAnyValue = true;
                 }
             }
-            return null;
+            return didAddAnyValue;
         }
         else if (typeof value === 'object' && value !== null) {
+            let didAddAnyValue = false;
             for (const [objKey, val] of Object.entries(value)) {
                 const nestedKey = objectNotation === 'dot' ? `${key}.${objKey}` : `${key}[${objKey}]`;
-                processValue(nestedKey, val);
+                didAddAnyValue = processValue(nestedKey, val) || didAddAnyValue;
             }
-            return null;
+            return didAddAnyValue;
         }
         else {
             if (ignoredValues.includes(value)) {
@@ -1547,11 +1550,11 @@ const withSuccessHandling = (options) => {
         type: QueryFeatureType.WITH_SUCCESS_HANDLING,
         fn: (context) => {
             nestedEffect(() => {
-                const response = context.state.response();
-                if (response === null)
+                const event = context.state.latestHttpEvent();
+                if (!event || event.type !== HttpEventType.Response)
                     return;
                 untracked(() => {
-                    options.handler(response);
+                    options.handler(context.state.response());
                 });
             }, { injector: context.deps.injector });
         },
@@ -1684,7 +1687,8 @@ const createQuerySnapshotFn = (options) => {
                 snapshotState.response.set(currentResponse);
                 if (currentLoading)
                     return;
-                if (!currentResponse && !currentError)
+                const hasCompletedResponse = currentLatestHttpEvent?.type === HttpEventType.Response;
+                if (!hasCompletedResponse && !currentError)
                     return;
                 // kill the effect once loading is done and we either have a response or an error
                 killEffectRef.destroy();
@@ -2084,7 +2088,6 @@ const createHttpRequest = (options) => {
         return true;
     };
     const updateState = (event) => {
-        currentEvent.set(event);
         switch (event.type) {
             case HttpEventType.Response:
                 {
@@ -2111,6 +2114,7 @@ const createHttpRequest = (options) => {
                 }
                 break;
         }
+        currentEvent.set(event);
     };
     const updateErrorState = (errorResponse) => {
         const errorRes = createQueryErrorResponse(errorResponse);