Skip to content
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

Add roles to tags and classification #18245

Draft
wants to merge 105 commits into
base: main
Choose a base branch
from
Draft

Add roles to tags and classification #18245

wants to merge 105 commits into from

Conversation

harshach
Copy link
Collaborator

Describe your changes:

Fixes

I worked on ... because ...

Type of change:

  • Bug fix
  • Improvement
  • New feature
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation

Checklist:

  • I have read the CONTRIBUTING document.
  • My PR title is Fixes <issue-number>: <short explanation>
  • I have commented on my code, particularly in hard-to-understand areas.
  • For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

Copy link
Contributor

The Java checkstyle failed.

Please run mvn spotless:apply in the root of your repository and commit the changes to this PR.
You can also use pre-commit to automate the Java code formatting.

You can install the pre-commit hooks with make install_test precommit_install.

aniketkatkar97 and others added 20 commits October 19, 2024 14:57
* added check to append reserve-sidebar class to body

* added application value as dependency to re-run in case application refetch

* fix the unit test
* fix import issue

* fix: added origin entity FQN to test case index

* feat: added aggregation support for bucket selector

* feat: moved aggregation builder to its own class

* style: ran java linting

* fix: typos

* style: ran java linting

* feat: implement aggregation builder graph

* feat: clean up existing aggregation implementation for graph builder

* style: ran java linting

* fix: clean up comment

* style: fix java linting

---------

Co-authored-by: Chirag Madlani <[email protected]>
* GEN-1322: API Entity - Remove Beta

* minor: add doc for the metadata pipeline

* api service refactor

* api service refactor backend changes

* add apiconnection in test service connection

* pytest fix

* fix java file formatting

* Fix casing of REST in ApiServiceRest.spec.ts

* Refactor REST to Rest in API classes

* minor change

* minor change

* minor change

* fix cashing for API to Api

* add playwright test for api service ingestion

* fix: playwright test

---------

Co-authored-by: harshsoni2024 <[email protected]>
* add max length validation for glossary term name

* add name validation
* MINOR: open links in new tab in DataAssetsHeader

* fix: make mlModelDetail.dashboard internal link
* chore(ui): pull persona as top level settings

* fix tests

* update breadcrumbs

* add missing type

* fix ui tests

* fix settings router tests
)

* fix activityFeed and DataInsight AUT playwright test

* minor improvement
pmbrull and others added 25 commits October 19, 2024 14:57
…#18236)

* update connections

* MINOR - Pass timeout in test connection and return TestConnectionStep

* format

* comments

* comments
* GEN-1166 - Improve Ingestion Workflow Error Summary

* fix test

* docs

* comments
* fix: custom properties folder name in generation to match expected package name (i.e. customProperties -- uppercase P)

* fix: allow non admin/bot to read profiler global config with the right permission

* style: ran java linting

* fix: custom properties import casing
* feat: added test case results to search reindex

* fix: failing typescript test case
* fix: added testSuite field back to lineage response

* style: fix style
* what's new in 1.5.7 doc

* minor fix around indentation

* Update whatsNewData.ts 

Update whatsNewData.ts -- remove enumWithDescriptions pointer from the list

* Added improvement for table-level, export-import

Added improvement for table-level, export-import

---------

Co-authored-by: Ashish Gupta <[email protected]>
Co-authored-by: sonika-shah <[email protected]>
* add query builder widget

* locales

* add unit tests

* add debounce for on change
…18310)

* fixed task deserialization in Airflow metadata ingestion

* fixed formatting

---------

Co-authored-by: Katarzyna Kałek <[email protected]>
* Minor: Search RBAC, fix condition evaluation for single policy multiple rules

* Minor: Search RBAC, fix condition evaluation for single policy multiple rules

* add test for complex policies

* Add viewAll policy to organization

---------

Co-authored-by: Mohit Yadav <[email protected]>
* MINOR - Return TestConnectionResult from test_connection fn

* MINOR - Return TestConnectionResult from test_connection fn
* fix: live index on test suite creation

* fix: make live indexing use entityInterface
…or a given entity type (#18248)

* Minor: Add an API to list all the properties and custom properties for a given entity type

* Minor: Add an API to list all the properties and custom properties for a given entity type

* add custom properties

* add custom properties

* add custom properties
…UI (#18242)

* Fix #18007: Disabled Classifications or Tags shouldn't be visible in UI

* added playwright test for disabled tags should not be visible while search

* replace testing tag to new generated one to avoid flakyness

* added test for checking tags are re-enabling it from disabled state

* fix the playwright test for the wrong column selector

---------

Co-authored-by: Ashish Gupta <[email protected]>
Copy link
Contributor

The Java checkstyle failed.

Please run mvn spotless:apply in the root of your repository and commit the changes to this PR.
You can also use pre-commit to automate the Java code formatting.

You can install the pre-commit hooks with make install_test precommit_install.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend safe to test Add this label to run secure Github workflows on PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.