Fix GetPoolTicker performance

Author: Damien LACHAUME / PALO-IT

mithril-explorer/__tests__/store.test.js

@@ -8,6 +8,7 @@ import {
 } from "../src/store/settingsSlice";
 import default_available_aggregators from "../src/aggregators-list";
 import {initStore} from "./helpers";
+import {poolsSlice} from "../src/store/poolsSlice";
 
 describe('Store Initialization', () => {
   it('init with settings initialState without local storage', () => {
@@ -19,6 +20,7 @@ describe('Store Initialization', () => {
   it('init with local storage saved state', () => {
     let aggregators = [...default_available_aggregators, "https://aggregator.test"];
     let expected = {
+      pools: poolsSlice.getInitialState(),
       settings: {
         ...settingsSlice.getInitialState(),
         selectedAggregator: aggregators.at(aggregators.length - 1),
@@ -36,6 +38,7 @@ describe('Store Initialization', () => {
     const initialAggregator = default_available_aggregators.at(1);
     let aggregators = [...default_available_aggregators, "https://aggregator.test"];
     let expected = {
+      pools: poolsSlice.getInitialState(),
       settings: {
         ...settingsSlice.getInitialState(),
         selectedAggregator: aggregators.at(aggregators.length - 1),

mithril-explorer/src/store/poolsSlice.js

@@ -1,16 +1,23 @@
-import {createAsyncThunk, createSlice} from "@reduxjs/toolkit";
-import {selectedAggregator} from "./settingsSlice";
+import {createAsyncThunk, createSelector, createSlice} from "@reduxjs/toolkit";
 
 export const poolsSlice = createSlice({
     name: 'pools',
-    initialState: {},
+    initialState: { list: [] },
     reducers: {},
     extraReducers: builder => builder.addCase(updatePoolsForAggregator.fulfilled, (state, action) => {
-        state[action.payload.aggregator] = {
-            date: action.payload.date,
-            network: action.payload.network,
-            pools: action.payload.pools,
-        };
+        let existing = poolsForAggregator(state, action.payload.aggregator);
+
+        if (existing) {
+            existing.date = action.payload.date;
+            existing.pools = action.payload.pools;
+        } else {
+            state.list.push({
+                aggregator: action.payload.aggregator,
+                date: action.payload.date,
+                network: action.payload.network,
+                pools: action.payload.pools,
+            });
+        }
     })
 });
 
@@ -27,12 +34,26 @@ export const updatePoolsForAggregator = createAsyncThunk('pools/updateForAggrega
         });
 });
 
-export const getPool = (state, aggregator, poolId) => {
-    const data = state.pools[aggregator];
+const poolsForAggregator = (state, aggregator) => {
+    return state.list.find(poolsData => poolsData.aggregator === aggregator);
+};
+
+export const getPool = createSelector([
+        state => state.pools,
+        (state, aggregator, poolId) => {
     return {
-        network: data?.network,
-        ...data?.pools.find(pool => pool.party_id === poolId)
+        aggregator: aggregator,
+        poolId: poolId
+    }},
+    ],
+    (pools, args) => {
+    const aggregator = poolsForAggregator(pools, args.aggregator);
+    const data = aggregator?.pools.find(pool => pool.party_id === args.poolId);
+
+    return {
+        network: aggregator?.network,
+        ...data,
     };
-};
+});
 
 export default poolsSlice.reducer;