Skip to content

Fix bug on holdings page #3613

Fix bug on holdings page

Fix bug on holdings page #3613

Triggered via pull request October 4, 2024 09:36
Status Failure
Total duration 1m 15s
Artifacts

centrifuge-app.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

3 errors and 11 warnings
build-app: centrifuge-app/src/components/InvestRedeem/InvestRedeemDrawer.tsx#L31
React Hook "React.useMemo" is called conditionally. React Hooks must be called in the exact same order in every component render
build-app: centrifuge-app/src/components/InvestRedeem/InvestRedeemDrawer.tsx#L54
React Hook "useDailyPoolStates" is called conditionally. React Hooks must be called in the exact same order in every component render
build-app
Process completed with exit code 1.
build-app
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@ac593985615ec2ede58e132d2e21d2b1cbd6127c, actions/setup-node@64ed1c7eab4cce3362f8c340dee64e5eaeef8f7c. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
build-app: centrifuge-app/src/components/Charts/PoolPerformanceChart.tsx#L174
React Hook React.useMemo has unnecessary dependencies: 'isSingleTranche' and 'todayPrice'. Either exclude them or remove the dependency array
build-app: centrifuge-app/src/components/Charts/PoolPerformanceChart.tsx#L212
React Hook React.useMemo has an unnecessary dependency: 'selectedTabIndex'. Either exclude it or remove the dependency array
build-app: centrifuge-app/src/components/Charts/PoolPerformanceChart.tsx#L482
Array.prototype.map() expects a return value from arrow function
build-app: centrifuge-app/src/components/InvestRedeem/InvestRedeemDrawer.tsx#L137
React Hook React.useMemo has a missing dependency: 'apy'. Either include it or remove the dependency array
build-app: centrifuge-app/src/components/PoolList.tsx#L54
React Hook React.useMemo has missing dependencies: 'cent' and 'centPoolsMetaDataById'. Either include them or remove the dependency array
build-app: centrifuge-app/src/components/PoolOverview/KeyMetrics.tsx#L100
React Hook useMemo has a missing dependency: 'pool.currency.decimals'. Either include it or remove the dependency array
build-app: centrifuge-app/src/components/PoolOverview/TrancheTokenCards.tsx#L31
The 'columnConfig' array makes the dependencies of useMemo Hook (at line 85) change on every render. Move it inside the useMemo callback. Alternatively, wrap the initialization of 'columnConfig' in its own useMemo() Hook
build-app: centrifuge-app/src/components/PoolOverview/TransactionHistory.tsx#L27
'getTransactionTypeStatus' is assigned a value but never used
build-app: centrifuge-app/src/components/PoolOverview/TransactionHistory.tsx#L201
'label' is assigned a value but never used
build-app: centrifuge-app/src/components/Report/BalanceSheet.tsx#L249
React Hook React.useEffect has a missing dependency: 'setReportData'. Either include it or remove the dependency array