Skip to content

Comments

fix: don't display "Datasets bulk" on deals without dataset bulk#115

Merged
PierreJeanjacquot merged 1 commit intomainfrom
fix/datasets-bulk-display
Feb 18, 2026
Merged

fix: don't display "Datasets bulk" on deals without dataset bulk#115
PierreJeanjacquot merged 1 commit intomainfrom
fix/datasets-bulk-display

Conversation

@PierreJeanjacquot
Copy link
Member

@PierreJeanjacquot PierreJeanjacquot commented Feb 18, 2026

The "Dataset" row was incorrectly displayed as "Datasets bulk" when a deal has no dataset or dataset bulk

Example with deal https://explorer.iex.ec/arbitrum-sepolia-testnet/deal/0xeb489bebd59399b245d1a15fe8710129f470177b71f2b1160d4ea704e6d744d6?dealTab=DETAILS (not bulk)

before after
image image

Note

Low Risk
Small UI-conditional change limited to deal details rendering; no security or data-handling logic is affected.

Overview
Fixes deal details rendering so the "Datasets bulk" placeholder is only shown when deal.dataset is null and the deal is marked as bulk, preventing incorrect UI on non-bulk deals without a dataset.

Written by Cursor Bugbot for commit 768e89c. This will update automatically on new commits. Configure here.

@vercel
Copy link

vercel bot commented Feb 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
explorer-v2 Ready Ready Preview, Comment Feb 18, 2026 8:55am

Request Review

@PierreJeanjacquot PierreJeanjacquot merged commit 7ba088f into main Feb 18, 2026
11 checks passed
@PierreJeanjacquot PierreJeanjacquot deleted the fix/datasets-bulk-display branch February 18, 2026 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants