feat: update filter query

src/api/accounts.test.ts

@@ -0,0 +1,23 @@
+import { print } from 'graphql';
+
+import {
+  GET_ACCOUNT_BY_ID,
+  GET_ACCOUNTS,
+  GET_FILTERED_ACCOUNTS
+} from './accounts';
+
+describe('account queries', () => {
+  it.each([
+    ['GetAccounts', GET_ACCOUNTS],
+    ['GetFilteredAccounts', GET_FILTERED_ACCOUNTS],
+    ['GetAccountById', GET_ACCOUNT_BY_ID]
+  ] as const)(
+    '%s reads has_mined_blocks instead of peeking minedBlocks',
+    (_name, document) => {
+      const query = print(document);
+
+      expect(query).toContain('has_mined_blocks');
+      expect(query).not.toContain('minedBlocks');
+    }
+  );
+});

src/api/accounts.tsx

@@ -24,9 +24,7 @@ const ACCOUNT_LIST_FIELDS = `
   is_high_security
   is_guardian
   is_multisig
-  minedBlocks(limit: 1) {
-    height
-  }
+  has_mined_blocks
 `;
 
 export const GET_ACCOUNTS = gql`
@@ -71,9 +69,7 @@ export const GET_ACCOUNT_BY_ID = gql`
       frozen
       reserved
       is_deposit_only
-      minedBlocks(limit: 1) {
-        height
-      }
+      has_mined_blocks
     }
     accountStats: account_stats_by_pk(id: $id) {
       total_cancelled_transfers

src/components/common/table-columns/ACCOUNT_COLUMNS.tsx

@@ -31,7 +31,7 @@ export const ACCOUNT_COLUMNS = [
         is_guardian: isGuardian,
         is_multisig: isMultisig,
         is_deposit_only: isDepositOnly,
-        minedBlocks
+        has_mined_blocks: hasMinedBlocks
       } = props.row.original;
       const hasFlags = isHighSec || isGuardian || isMultisig;
       const addressLabel = (
@@ -40,7 +40,7 @@ export const ACCOUNT_COLUMNS = [
           isGuardian={isGuardian}
           isMultisig={isMultisig}
           isDepositOnly={isDepositOnly}
-          hasMinedBlocks={minedBlocks.length > 0}
+          hasMinedBlocks={hasMinedBlocks}
         />
       );
 

src/components/features/account-details/account-information/AccountInformation.tsx

@@ -72,7 +72,7 @@ export const AccountInformation: React.FC<AccountInformationProps> = ({
       isGuardian,
       isMultisig,
       isDepositOnly: account?.is_deposit_only ?? null,
-      hasMinedBlocks: (account?.minedBlocks.length ?? 0) > 0
+      hasMinedBlocks: account?.has_mined_blocks ?? false
     }
   ];
 

src/schemas/account.ts

@@ -5,16 +5,12 @@ export type Account = Pick<
   'id' | 'free' | 'frozen' | 'reserved' | 'is_deposit_only'
 >;
 
-/** At most one mined block, fetched only to tell whether the account has ever mined. */
-export interface AccountMinedBlocksPeek {
-  minedBlocks: { height: number }[];
-}
-
 /** Listing row: flags are denormalized onto `account` by the indexer (no nested event lookups). */
-export interface AccountListItem extends Account, AccountMinedBlocksPeek {
+export interface AccountListItem extends Account {
   is_high_security: boolean;
   is_guardian: boolean;
   is_multisig: boolean;
+  has_mined_blocks: boolean;
 }
 
 export interface AccountListPageResponse {
@@ -32,7 +28,7 @@ export interface AccountStats {
 }
 
 export interface AccountResponse {
-  account: (Account & AccountMinedBlocksPeek) | null;
+  account: (Account & Pick<gql.Account, 'has_mined_blocks'>) | null;
   accountStats: AccountStats;
   multisig?: { id: string } | null;
   beneficiaries: {

src/utils/account-type-filter.test.ts

@@ -27,15 +27,15 @@ describe('accountTypeWhere', () => {
     expect(accountTypeWhere('potential_encrypted')).toEqual({
       ...STANDARD,
       is_deposit_only: { _eq: true },
-      _not: { minedBlocks: {} }
+      has_mined_blocks: { _eq: false }
     });
   });
 
   it('matches unflagged deposit-only accounts with mined blocks as encrypted', () => {
     expect(accountTypeWhere('encrypted')).toEqual({
       ...STANDARD,
       is_deposit_only: { _eq: true },
-      minedBlocks: {}
+      has_mined_blocks: { _eq: true }
     });
   });
 

src/utils/account-type-filter.ts

@@ -30,13 +30,13 @@ export function accountTypeWhere(type: AccountType): Account_Bool_Exp {
       return {
         ...STANDARD_ACCOUNT,
         is_deposit_only: { _eq: true },
-        _not: { minedBlocks: {} }
+        has_mined_blocks: { _eq: false }
       };
     case 'encrypted':
       return {
         ...STANDARD_ACCOUNT,
         is_deposit_only: { _eq: true },
-        minedBlocks: {}
+        has_mined_blocks: { _eq: true }
       };
     case 'high_security':
       return { is_high_security: { _eq: true } };