Skip to content

improvement(ui/ingestion): implement feedback on ingestion run details page#16045

Open
purnimagarg1 wants to merge 1 commit intomasterfrom
pg--cat-1243-feedback-run-details-page
Open

improvement(ui/ingestion): implement feedback on ingestion run details page#16045
purnimagarg1 wants to merge 1 commit intomasterfrom
pg--cat-1243-feedback-run-details-page

Conversation

@purnimagarg1
Copy link
Collaborator

Linear ticket:
https://linear.app/acryl-data/issue/CAT-1243/uiux-feedback-on-ingestion-run-llm-page

Description:

Bringing this PR back to OSS

  • Only show scrollbars on ingestion logs and ingestion recipe when hovered

Screenshots:

Scrollbar on hover:
image

@github-actions github-actions bot added the product PR or Issue related to the DataHub UI/UX label Feb 2, 2026
@datahub-cyborg datahub-cyborg bot added the needs-review Label for PRs that need review from a maintainer. label Feb 2, 2026
@alwaysmeticulous
Copy link

alwaysmeticulous bot commented Feb 2, 2026

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

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

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

@codecov
Copy link

codecov bot commented Feb 2, 2026

Bundle Report

Changes will increase total bundle size by 263 bytes (0.0%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
datahub-react-web-esm 29.55MB 263 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 263 bytes 19.38MB 0.0%

Files in assets/index-*.js:

  • ./src/app/ingestV2/executions/components/BaseTab.tsx → Total Size: 1.12kB

@codecov
Copy link

codecov bot commented Feb 2, 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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review Label for PRs that need review from a maintainer. product PR or Issue related to the DataHub UI/UX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant