https://datahubproject.io logo
Join Slack
Thread
#getting-started
    h

    high-hospital-85984

    1 year ago
    Copy to Clipboard
    It seem’s like the frontend/OwnerViewDao only supports CorpUsers as owners, and not groups, even though the 
    Owner
    -model states that owners can be groups. Is this a bug or by design?
    g

    green-football-43791

    1 year ago
    Copy to Clipboard
    I'm not very familiar with this class but I agree, it does seem like a bug. I would expect to see another line below 
    corpGroups = _corpGroups.batchGet(getGroupUrns(ownership))
    Copy to Clipboard
    I bet this class just wasn't extended to also support groups. I'd encourage you to put the fix up if you'd like that functionality
    h

    high-hospital-85984

    1 year ago
    Copy to Clipboard
    Sure. The short term solution is of course to fix the comment in 
    Owner
    so that users don’t end up breaking their UI.
    Copy to Clipboard
    I created this issue so that we don’t forget about it: https://github.com/linkedin/datahub/issues/2138
    g

    green-football-43791

    1 year ago
    Copy to Clipboard
    perfect- here's a change to update the comment in pdl:
    Copy to Clipboard
    https://github.com/linkedin/datahub/pull/2139
View count: 1
Join thread in Slack