Added check to not cache data from subgraph for disabled networks

Author: 0xVoronov

packages/apps/dashboard/server/src/modules/details/details.module.ts

@@ -3,9 +3,10 @@ import { Module } from '@nestjs/common';
 import { DetailsService } from './details.service';
 import { DetailsController } from './details.controller';
 import { HttpModule } from '@nestjs/axios';
+import { StatsModule } from '../stats/stats.module';
 
 @Module({
-  imports: [HttpModule],
+  imports: [HttpModule, StatsModule],
   controllers: [DetailsController],
   providers: [DetailsService],
 })

packages/apps/dashboard/server/src/modules/details/details.service.ts

@@ -1,5 +1,4 @@
 import { plainToInstance } from 'class-transformer';
-import { MainnetsId } from '../../common/utils/constants';
 import { BadRequestException, Injectable, Logger } from '@nestjs/common';
 import {
   ChainId,
@@ -21,11 +20,13 @@ import { firstValueFrom } from 'rxjs';
 import { HMToken__factory } from '@human-protocol/core/typechain-types';
 import { ethers } from 'ethers';
 import { NetworkConfigService } from '../../common/config/network-config.service';
+import { StatsService } from '../stats/stats.service';
 
 @Injectable()
 export class DetailsService {
   private readonly logger = new Logger(DetailsService.name);
   constructor(
+    private readonly statsService: StatsService,
     private readonly configService: EnvironmentConfigService,
     private readonly httpService: HttpService,
     private readonly networkConfig: NetworkConfigService,
@@ -149,9 +150,7 @@ export class DetailsService {
 
   public async getBestLeadersByRole(chainId?: ChainId): Promise<LeaderDto[]> {
     const chainIds = !chainId
-      ? (Object.values(MainnetsId).filter(
-          (value) => typeof value === 'number',
-        ) as number[])
+      ? await this.statsService.getAvailableNetworks()
       : [chainId];
 
     const leadersByRole: { [role: string]: LeaderDto } = {};
@@ -187,9 +186,7 @@ export class DetailsService {
 
   public async getAllLeaders(chainId?: ChainId): Promise<LeaderDto[]> {
     const chainIds = !chainId
-      ? (Object.values(MainnetsId).filter(
-          (value) => typeof value === 'number',
-        ) as number[])
+      ? await this.statsService.getAvailableNetworks()
       : [chainId];
 
     const allLeaders: LeaderDto[] = [];

packages/apps/dashboard/server/src/modules/stats/stats.module.ts

@@ -8,6 +8,7 @@ import { StorageModule } from '../storage/storage.module';
 @Module({
   imports: [HttpModule, StorageModule],
   controllers: [StatsController],
+  exports: [StatsService],
   providers: [StatsService],
 })
 export class StatsModule {}

packages/apps/dashboard/server/src/modules/stats/stats.service.ts

@@ -180,9 +180,8 @@ export class StatsService implements OnModuleInit {
       totalHolders: 0,
       totalTransactions: 0,
     };
-    for (const network of Object.values(MainnetsId).filter(
-      (value) => typeof value === 'number',
-    ) as number[]) {
+    const availableNetworks = await this.getAvailableNetworks();
+    for (const network of availableNetworks) {
       const statisticsClient = new StatisticsClient(NETWORKS[network]);
       const generalStats = await statisticsClient.getHMTStatistics();
       aggregatedStats.totalHolders += generalStats.totalHolders;
@@ -219,13 +218,11 @@ export class StatsService implements OnModuleInit {
     const dailyData: Record<string, CachedHMTData> = {};
     const monthlyData: Record<string, CachedHMTData> = {};
 
+    const availableNetworks = await this.getAvailableNetworks();
+
     // Fetch daily data for each network
     await Promise.all(
-      (
-        Object.values(MainnetsId).filter(
-          (value) => typeof value === 'number',
-        ) as number[]
-      ).map(async (network) => {
+      availableNetworks.map(async (network) => {
         const statisticsClient = new StatisticsClient(NETWORKS[network]);
         let skip = 0;
         let fetchedRecords: DailyHMTData[] = [];