fix(ownership): Corrects validation of ownership type and makes it consistent across graphQL calls #9044

pedro93 · 2023-10-18T18:38:33Z

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

…nsistent across graphQL calls

...raphql-core/src/main/java/com/linkedin/datahub/graphql/resolvers/mutate/util/OwnerUtils.java

...l-core/src/test/java/com/linkedin/datahub/graphql/resolvers/owner/AddOwnersResolverTest.java

jjoyce0510

Small comments, otherwise LGTM

fix(ownership): Corrects validation of ownership type and makes it co…

042e7ed

…nsistent across graphQL calls

pedro93 added the release-0.12.0 PRs targeted for Release 0.12.0 label Oct 18, 2023

pedro93 assigned eboneil Oct 18, 2023

github-actions bot added the product PR or Issue related to the DataHub UI/UX label Oct 18, 2023

vercel bot deployed to Preview October 18, 2023 20:06 View deployment

Fix compile errors

f1eee48

pedro93 requested a review from jjoyce0510 October 18, 2023 20:19

eboneil added 2 commits October 18, 2023 13:40

add unit tests

e614f99

checkstyle

cd1e214

vercel bot deployed to Preview October 18, 2023 22:34 View deployment

jjoyce0510 reviewed Oct 18, 2023

View reviewed changes

...raphql-core/src/main/java/com/linkedin/datahub/graphql/resolvers/mutate/util/OwnerUtils.java Show resolved Hide resolved

jjoyce0510 reviewed Oct 18, 2023

View reviewed changes

...l-core/src/test/java/com/linkedin/datahub/graphql/resolvers/owner/AddOwnersResolverTest.java Show resolved Hide resolved

jjoyce0510 reviewed Oct 18, 2023

View reviewed changes

...l-core/src/test/java/com/linkedin/datahub/graphql/resolvers/owner/AddOwnersResolverTest.java Show resolved Hide resolved

jjoyce0510 approved these changes Oct 18, 2023

View reviewed changes

eboneil merged commit 269c4ea into master Oct 19, 2023
35 of 37 checks passed

eboneil deleted the ps-fix-owner-input branch October 19, 2023 00:43

maggiehays added the hacktoberfest-accepted Acceptance for hacktoberfest https://hacktoberfest.com/participation/ label Oct 26, 2023

This pull request was closed.