Skip to content

fix(privileges) Add Term Groups as targetable entities for privileges #5806

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Merged
merged 1 commit into from
Sep 1, 2022
Merged

fix(privileges) Add Term Groups as targetable entities for privileges #5806

merged 1 commit into from
Sep 1, 2022

Conversation

chriscollins3456
Copy link
Collaborator

In response to this issue.

Adds Glossary Nodes (aka Term Groups) to be targetable entities for adding / editing metadata privileges.

Checklist

  • The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
  • Links to related issues (if applicable)
  • Tests for the changes have been added/updated (if applicable)
  • Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
  • For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

@github-actions
Copy link

github-actions bot commented Sep 1, 2022

Unit Test Results (build & test)

517 tests  ±0   517 ✔️ ±0   8m 36s ⏱️ -35s
121 suites ±0       0 💤 ±0 
121 files   ±0       0 ❌ ±0 

Results for commit ba99d4a. ± Comparison against base commit 5bf5fc2.

@jjoyce0510 jjoyce0510 merged commit b0fc67b into datahub-project:master Sep 1, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@gabe-lyons gabe-lyons gabe-lyons approved these changes

@jjoyce0510 jjoyce0510 jjoyce0510 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@chriscollins3456 @gabe-lyons @jjoyce0510