♻️ refactor: remove unused onClick and create a separate pagination t…

…ype.
LerianStudio · Dec 16, 2024 · b0974cf · b0974cf
1 parent 4b0a866
commit b0974cf
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/src/app/(routes)/ledgers/ledgers-data-table.tsx b/src/app/(routes)/ledgers/ledgers-data-table.tsx
@@ -199,7 +199,7 @@ const LedgerRow: React.FC<LedgerRowProps> = ({
                   className="h-auto w-max p-2"
                   data-testid="actions"
                 >
-                  <MoreVertical size={16} onClick={() => {}} />
+                  <MoreVertical size={16} />
                 </Button>
               </DropdownMenuTrigger>
               <DropdownMenuContent align="end">

diff --git a/src/client/ledgers.ts b/src/client/ledgers.ts
@@ -6,6 +6,7 @@ import {
   patchFetcher,
   postFetcher
 } from '@/lib/fetcher'
+import { PaginationRequest } from '@/types/pagination-request-type'
 import {
   useMutation,
   UseMutationOptions,
@@ -51,7 +52,7 @@ const useListLedgers = ({
   enabled = true,
   limit = 10,
   page = 1
-}: UseListLedgersProps & { limit?: number; page?: number }) => {
+}: UseListLedgersProps & PaginationRequest) => {
   return useQuery<PaginationDto<LedgerResponseDto>>({
     queryKey: ['ledgers', organizationId, { limit, page }],
     queryFn: getFetcher(

diff --git a/src/types/pagination-request-type.ts b/src/types/pagination-request-type.ts
@@ -0,0 +1,4 @@
+export type PaginationRequest = {
+  limit?: number
+  page?: number
+}