Skip to content

(fix: asset downloads) Asset download queries do not apply filters and download all assets#16075

Merged
ani-malgari merged 1 commit intomasterfrom
fix/cus-7518_download-assets-clean
Feb 5, 2026
Merged

(fix: asset downloads) Asset download queries do not apply filters and download all assets#16075
ani-malgari merged 1 commit intomasterfrom
fix/cus-7518_download-assets-clean

Conversation

@ani-malgari
Copy link
Contributor

For asset downloads on any given page, we do not apply filters and download all available assets on the platform. This PR fixes the query on the frontend.

@github-actions github-actions bot added product PR or Issue related to the DataHub UI/UX community-contribution PR or Issue raised by member(s) of DataHub Community labels Feb 4, 2026
@alwaysmeticulous
Copy link

alwaysmeticulous bot commented Feb 4, 2026

✅ Meticulous spotted 0 visual differences across 919 screens tested: view results.

Meticulous evaluated ~8 hours of user flows against your PR.

Expected differences? Click here. Last updated for commit d3a112f. This comment will update as new commits are pushed.

@datahub-cyborg datahub-cyborg bot added the needs-review Label for PRs that need review from a maintainer. label Feb 4, 2026
@codecov
Copy link

codecov bot commented Feb 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@codecov
Copy link

codecov bot commented Feb 4, 2026

Bundle Report

Changes will decrease total bundle size by 12 bytes (-0.0%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
datahub-react-web-esm 29.56MB -12 bytes (-0.0%) ⬇️

Affected Assets, Files, and Routes:

view changes for bundle: datahub-react-web-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-*.js -12 bytes 19.39MB -0.0%

Files in assets/index-*.js:

  • ./src/app/entity/shared/components/styled/search/EmbeddedListSearch.tsx → Total Size: 7.44kB

  • ./src/app/entityV2/shared/components/styled/search/EmbeddedListSearch.tsx → Total Size: 9.85kB

@datahub-cyborg datahub-cyborg bot added merge-pending-ci A PR that has passed review and should be merged once CI is green. and removed needs-review Label for PRs that need review from a maintainer. labels Feb 5, 2026
@ani-malgari ani-malgari merged commit 34193ae into master Feb 5, 2026
51 checks passed
@ani-malgari ani-malgari deleted the fix/cus-7518_download-assets-clean branch February 5, 2026 19:46
neildsouth pushed a commit to National-Digital-Twin/ndt-data-catalogue that referenced this pull request Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution PR or Issue raised by member(s) of DataHub Community merge-pending-ci A PR that has passed review and should be merged once CI is green. product PR or Issue related to the DataHub UI/UX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants