Skip to content

Commit 5907fe3

Browse files
authored
Merge pull request #1016 from PayButton/fix/button-detail-no-txs
fix: button detail for 0 txs
2 parents c1fcffc + 59e875a commit 5907fe3

3 files changed

Lines changed: 10 additions & 14 deletions

File tree

components/TableContainer/TableContainerGetter.tsx

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import { useTable, usePagination } from 'react-table'
33
import { DEFAULT_EMPTY_TABLE_MESSAGE } from 'constants/index'
44
import style from './table-container.module.css'
55

6-
interface DataGetterReturn {
6+
export interface DataGetterReturn {
77
data: any
88
totalCount: number
99
}
@@ -35,7 +35,7 @@ const TableContainer = ({ columns, dataGetter, opts, ssr, tableRefreshCount, emp
3535
const [pageCount, setPageCount] = useState(0)
3636
const [loading, setLoading] = useState(true)
3737
const emptyMessageDisplay = emptyMessage ?? DEFAULT_EMPTY_TABLE_MESSAGE
38-
const [hiddenColumns, setHiddenColumns] = useState({})
38+
const [hiddenColumns, setHiddenColumns] = useState<Record<string, boolean>>({})
3939

4040
const triggerSort = (column: any): void => {
4141
if (column.disableSortBy === true || hiddenColumns[column.id]) return
@@ -49,9 +49,9 @@ const TableContainer = ({ columns, dataGetter, opts, ssr, tableRefreshCount, emp
4949
}
5050
gotoPage(0)
5151
}
52-
52+
5353
const toggleColumn = (id: any): void => {
54-
setHiddenColumns((prev) => ({ ...prev, [id]: !prev[id]}))
54+
setHiddenColumns((prev) => ({ ...prev, [id]: !prev[id] }))
5555
}
5656

5757
const {
@@ -122,12 +122,12 @@ const TableContainer = ({ columns, dataGetter, opts, ssr, tableRefreshCount, emp
122122
<th {...column.getHeaderProps()} style={column.disableSortBy === true ? null : { cursor: 'pointer' }} onClick={() => { triggerSort(column) }}>
123123
<div>
124124
{column.render('Header')}
125-
{column.shrinkable && (
125+
{column.shrinkable === true && (
126126
<span onClick={() => toggleColumn(column.id)} style={{ cursor: 'pointer' }}>
127-
{hiddenColumns[column.id] ? <div style= {{marginRight: '5px'}} className='table-arrow-right' /> : <div style= {{marginRight: '5px'}} className='table-sort-arrow-down' />}
127+
{hiddenColumns[column.id] ? <div style= {{ marginRight: '5px' }} className='table-arrow-right' /> : <div style= {{ marginRight: '5px' }} className='table-sort-arrow-down' />}
128128
</span>
129129
)}
130-
{!column.shrinkable && generateSortingIndicator(column)}
130+
{column.shrinkable !== true && generateSortingIndicator(column)}
131131
</div>
132132
</th>
133133
))}
@@ -143,7 +143,7 @@ const TableContainer = ({ columns, dataGetter, opts, ssr, tableRefreshCount, emp
143143
return (
144144
<tr {...row.getRowProps()}>
145145
{row.cells.map((cell: any) =>
146-
hiddenColumns[cell.column.id] ? <td> </td> : <td {...cell.getCellProps()}>{cell.render('Cell')}</td>
146+
hiddenColumns[cell.column.id] ? <td> </td> : <td {...cell.getCellProps()}>{cell.render('Cell')}</td>
147147
)}
148148
</tr>
149149
)

components/Transaction/PaybuttonTransactions.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import Plus from 'assets/plus.png'
99
import Pencil from 'assets/pencil.png'
1010
import FileText from 'assets/file-text.png'
1111

12-
import TableContainerGetter from '../TableContainer/TableContainerGetter'
12+
import TableContainerGetter, { DataGetterReturn } from '../TableContainer/TableContainerGetter'
1313
import { compareNumericString } from 'utils/index'
1414
import moment from 'moment-timezone'
1515
import { XEC_TX_EXPLORER_URL, BCH_TX_EXPLORER_URL } from 'constants/index'
@@ -26,7 +26,7 @@ interface IProps {
2626
timezone: string
2727
}
2828

29-
function fetchTransactionsByPaybuttonId (paybuttonId: string): Function {
29+
function fetchTransactionsByPaybuttonId (paybuttonId: string): (page: number, pageSize: number, orderBy: string, orderDesc: boolean) => Promise<DataGetterReturn> {
3030
return async (page: number, pageSize: number, orderBy: string, orderDesc: boolean) => {
3131
const response = await fetch(`/api/paybutton/transactions/${paybuttonId}?page=${page}&pageSize=${pageSize}&orderBy=${orderBy}&orderDesc=${String(orderDesc)}`, {
3232
headers: {

services/transactionService.ts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -587,10 +587,6 @@ export async function fetchTransactionsByPaybuttonIdWithPagination (
587587
orderDesc,
588588
networkIds)
589589

590-
if (transactions.length === 0) {
591-
throw new Error(RESPONSE_MESSAGES.NO_TRANSACTION_FOUND_404.message)
592-
}
593-
594590
return transactions
595591
}
596592

0 commit comments

Comments
 (0)