Merge pull request #1258 from o1-labs/fix/actions-ordering

fix: remove reversing fetched actions
o1-labs · Nov 21, 2023 · dac55a5 · dac55a5
2 parents ac75360 + b91b033
commit dac55a5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -44,6 +44,10 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
 - `Gadgets.and()`, new provable method to support bitwise and. https://github.com/o1-labs/o1js/pull/1193
 - `Gadgets.multiRangeCheck()` and `Gadgets.compactMultiRangeCheck()`, two building blocks for non-native arithmetic with bigints of size up to 264 bits. https://github.com/o1-labs/o1js/pull/1216
 
+### Fixed
+
+- Removed array reversal of fetched actions, since they are returned in the correct order. https://github.com/o1-labs/o1js/pull/1258
+
 ## [0.14.0](https://github.com/o1-labs/o1js/compare/045faa7...e8e7510e1)
 
 ### Breaking changes

diff --git a/src/lib/fetch.ts b/src/lib/fetch.ts
@@ -951,10 +951,8 @@ async function fetchActions(
       break;
     }
   }
-  // Archive Node API returns actions in the latest order, so we reverse the array to get the actions in chronological order.
-  fetchedActions.reverse();
-  let actionsList: { actions: string[][]; hash: string }[] = [];
 
+  let actionsList: { actions: string[][]; hash: string }[] = [];
   // correct for archive node sending one block too many
   if (
     fetchedActions.length !== 0 &&