Skip to content

[WAIT FOR CI] feat(structuredProperties): add new property to hide properties with emty value #14870

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

[WAIT FOR CI] feat(structuredProperties): add new property to hide properties with emty value #14870

wants to merge 1 commit into from

Conversation

v-tarasevich-blitz-brain
Copy link
Contributor

This PR adds new property to settings of Structured Property to hide properties in asset sidebar when their value is empty

@github-actions github-actions bot added the product PR or Issue related to the DataHub UI/UX label Sep 26, 2025
@v-tarasevich-blitz-brain v-tarasevich-blitz-brain changed the title feat(structuredProperties): add option to hide properties with emty value [WAIT FOR CI] feat(structuredProperties): add option to hide properties with emty value Sep 26, 2025
@codecov Codecov
Copy link

codecov bot commented Sep 26, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 85.71429% with 2 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...edproperties/UpdateStructuredPropertyResolver.java 77.77% 0 Missing and 2 partials ⚠️

📢 Thoughts on this report? Let us know!

@codecov Codecov
Copy link

codecov bot commented Sep 26, 2025

Bundle Report

Bundle size has no change ✅

@v-tarasevich-blitz-brain v-tarasevich-blitz-brain changed the title [WAIT FOR CI] feat(structuredProperties): add option to hide properties with emty value [WIP] feat(structuredProperties): add option to hide properties with emty value Sep 26, 2025
@v-tarasevich-blitz-brain v-tarasevich-blitz-brain changed the title [WIP] feat(structuredProperties): add option to hide properties with emty value [WIP] feat(structuredProperties): add new property to hide properties with emty value Sep 29, 2025
@v-tarasevich-blitz-brain v-tarasevich-blitz-brain force-pushed the vt--ch-801-structured-properties--add-hide-when-empty-to-models branch from 1f6a4c6 to a21192d Compare September 29, 2025 10:05
@v-tarasevich-blitz-brain v-tarasevich-blitz-brain changed the title [WIP] feat(structuredProperties): add new property to hide properties with emty value feat(structuredProperties): add new property to hide properties with emty value Sep 29, 2025
@v-tarasevich-blitz-brain v-tarasevich-blitz-brain marked this pull request as ready for review September 29, 2025 13:33
@datahub-cyborg datahub-cyborg bot added the needs-review Label for PRs that need review from a maintainer. label Sep 29, 2025
@v-tarasevich-blitz-brain v-tarasevich-blitz-brain marked this pull request as draft September 29, 2025 14:59
@v-tarasevich-blitz-brain v-tarasevich-blitz-brain force-pushed the vt--ch-801-structured-properties--add-hide-when-empty-to-models branch from 7a8f8bc to 97b7897 Compare September 29, 2025 15:34
@v-tarasevich-blitz-brain v-tarasevich-blitz-brain changed the title feat(structuredProperties): add new property to hide properties with emty value [WAIT FOR CI] feat(structuredProperties): add new property to hide properties with emty value Sep 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@v-tarasevich-blitz-brain