-
Notifications
You must be signed in to change notification settings - Fork 320
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Minor UI enhancements #2727
Minor UI enhancements #2727
Conversation
# Conflicts: # web/src/components/datasets/DatasetDetailPage.tsx # web/src/store/sagas/index.ts
✅ Deploy Preview for peppy-sprite-186812 canceled.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2727 +/- ##
=========================================
Coverage 84.45% 84.45%
Complexity 1416 1416
=========================================
Files 251 251
Lines 6447 6447
Branches 291 291
=========================================
Hits 5445 5445
Misses 850 850
Partials 152 152 ☔ View full report in Codecov by Sentry. |
"minor" changes for 8000 lines? 😮 |
This is just on top of #2725, so it's minor after that. |
# Conflicts: # web/src/components/datasets/DatasetDetailPage.tsx # web/src/routes/column-level/ColumnLevel.tsx # web/src/routes/column-level/ColumnLevelDrawer.tsx # web/src/routes/column-level/ColumnLineageColumnNode.tsx # web/src/routes/column-level/layout.ts # web/src/routes/datasets/Datasets.tsx # web/src/routes/table-level/ActionBar.tsx # web/src/routes/table-level/TableLevel.tsx # web/src/routes/table-level/TableLineageDatasetNode.tsx # web/src/routes/table-level/TableLineageJobNode.tsx # web/src/routes/table-level/layout.ts
Fixed edge to non-existent node issue. .
# Conflicts: # web/src/components/datasets/DatasetInfo.tsx # web/src/components/datasets/DatasetTags.tsx
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 💯 🥇
Problem
This is a followup to the larger lineage update PR #2725 that fixes some minor application consistency and styling issues.
Solution
One-line summary: Hygienic cleanup of project
Checklist
CHANGELOG.md
(Depending on the change, this may not be necessary)..sql
database schema migration according to Flyway's naming convention (if relevant)