feat(UI): disable access management ui when no roles are linked to entity

[githendrik]

Disable access management ui when no roles are linked to entity

See https://datahubspace.slack.com/archives/CV2UXSE9L/p1704699920392329 for reference.

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

[chriscollins3456]

looks reasonable! quick check to see if you'd prefer to do the check if there's any roles at all for the dataset.

also - do you think it would make more sense to hide the tab entirely if there's no roles associated with it? i could go either way

[githendrik]

also - do you think it would make more sense to hide the tab entirely if there's no roles associated with it? i could go either way

I don’t mind either way :) would you want a 2nd (or 3rd) opinion on this? If so, feel free to @

Otherwise I’d probably keep it as-is. Wdyt?

[githendrik]

I guess since there was no veto regarding disabling the tab instead of hiding it, I'd keep it as-is? Then this behaves the same as e.g. the Validation tab

[githendrik]

Unfortunately Cypress seems to have some issue currently:

URL: https://download.cypress.io/desktop/12.5.1?platform=linux&arch=x64 Error: Failed downloading the Cypress binary. Response code: 502 Response message: Bad Gateway

[githendrik]

any updates? :)

[chriscollins3456]

let's make the one comment change and then we'll be good to merge! sorry for the delay on this one

[chriscollins3456]

nice!