Skip to content

Commit ed47e93

Browse files
committed
[BLD-338] Dashboard: Fix NFTs table owner row not showing data (#8120)
<!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR introduces a temporary change to the `table.tsx` file in the NFTs component of the dashboard. It sets the `useIndexer` property to `false`, indicating a workaround until a future fix is implemented. ### Detailed summary - Added `useIndexer: false` to the settings in `table.tsx`. - Marked the addition as temporary with a comment: `// TEMPORARY, TODO: Remove when this is fixed in insight`. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Bug Fixes** * Improved reliability of NFT lists in the dashboard, reducing intermittent errors and missing items when viewing NFTs for specific contracts. * Ensures more consistent, up-to-date NFT data is displayed during browsing and refreshes. * Enhances stability of data loading in environments with inconsistent indexing, minimizing failed loads and retries. * No visual/UI changes; behavior is more dependable without altering the interface or workflows. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent 1f7cda6 commit ed47e93

File tree

1 file changed

+1
-0
lines changed
  • apps/dashboard/src/app/(app)/(dashboard)/(chain)/[chain_id]/[contractAddress]/nfts/components

1 file changed

+1
-0
lines changed

apps/dashboard/src/app/(app)/(dashboard)/(chain)/[chain_id]/[contractAddress]/nfts/components/table.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,7 @@ export function NFTGetAllTable({
5858
includeOwners: true,
5959
start: currentPage * pageSize,
6060
tokenByIndex,
61+
useIndexer: false, // TEMPORARY, TODO: Remove when this is fixed in insight
6162
},
6263
);
6364

0 commit comments

Comments
 (0)