@gooddata/sdk-ui-ext 11.52.0-alpha.6 → 11.52.0

package/esm/share/useObjectShareDialog.js

@@ -1,5 +1,8 @@
 // (C) 2026 GoodData Corporation
 import { useCallback, useEffect, useState } from "react";
+import { useIntl } from "react-intl";
+import { objectShareMessages } from "./messages.js";
+import { changesEffectiveLevel, removalChangesEffectiveLevel } from "./objectShareController.helpers.js";
 import { useObjectShareController } from "./useObjectShareController.js";
 /**
  * Owns everything {@link ObjectShareDialog} needs to render for one dialog session:
@@ -10,6 +13,7 @@ import { useObjectShareController } from "./useObjectShareController.js";
  * @internal
  */
 export function useObjectShareDialog({ target, onClose, onSummaryChange, labels, labelsLoading, labelsError, }) {
+    const intl = useIntl();
     const { state, actions } = useObjectShareController(target, { labels, labelsError, labelsLoading });
     const [pendingSelfChange, setPendingSelfChange] = useState(undefined);
     // Synchronize the displayed access summary out to the consumer — the legitimate
@@ -31,7 +35,12 @@ export function useObjectShareDialog({ target, onClose, onSummaryChange, labels,
     // classified it as self-managed — those changes route through the confirm.
     const onRowPermissionChange = useCallback((grantee, level) => {
         if (state.selfManagedGranteeId === grantee.id) {
-            if (level !== grantee.level) {
+            // Stage the confirm only for a pick that would really move the level the
+            // user effectively holds. Comparing against the displayed level instead
+            // opened "Restrict your access?" for a pick under an inherited floor, which
+            // the controller then refuses — the user would confirm a restriction that
+            // never happens. Same predicate the controller applies.
+            if (changesEffectiveLevel(grantee, level)) {
                 setPendingSelfChange({ granteeId: grantee.id, level });
             }
             return;
@@ -39,7 +48,11 @@ export function useObjectShareDialog({ target, onClose, onSummaryChange, labels,
         void actions.changePermissionLevel(grantee.id, level);
     }, [state.selfManagedGranteeId, actions]);
     const onRowRemove = useCallback((grantee) => {
-        if (state.selfManagedGranteeId === grantee.id) {
+        // Stage the confirm only when the removal can lower the level the user
+        // effectively holds — under an inherited floor the revoke removes the local
+        // grant but restricts nothing, so the "Restrict your access?" warning would
+        // promise a restriction that never happens. Same policy as the picks above.
+        if (state.selfManagedGranteeId === grantee.id && removalChangesEffectiveLevel(grantee)) {
             setPendingSelfChange({ granteeId: grantee.id, level: "none" });
             return;
         }
@@ -60,6 +73,11 @@ export function useObjectShareDialog({ target, onClose, onSummaryChange, labels,
     }, [pendingSelfChange, actions]);
     const onSelfRestrictCancel = useCallback(() => setPendingSelfChange(undefined), []);
     const rowDisabledLevels = useCallback((grantee) => state.selfManagedGranteeId === grantee.id ? state.selfManagedDisabledLevels : undefined, [state.selfManagedGranteeId, state.selfManagedDisabledLevels]);
+    // Levels below an inherited one are deliberately NOT disabled: lowering the
+    // direct grant still reduces the effective level whenever it sits above what is
+    // inherited (direct EDIT under an inherited SHARE → picking VIEW yields SHARE).
+    // The row's warning badge is what explains the inherited floor.
+    const isRowRemoveDisabled = useCallback((grantee) => grantee.directLevel === undefined, []);
     const isMutable = state.status === "success" && state.labelsResolved;
     const isLoading = state.status === "loading" || state.labelsInitializing;
     return {
@@ -73,6 +91,8 @@ export function useObjectShareDialog({ target, onClose, onSummaryChange, labels,
         isLoading,
         workspaceDisabledLevels: state.workspaceDisabledLevels,
         rowDisabledLevels,
+        isRowRemoveDisabled,
+        rowRemoveDisabledTooltip: intl.formatMessage(objectShareMessages.granteeRemoveInherited),
         onClose,
         onRowPermissionChange,
         onRowRemove,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gooddata/sdk-ui-ext",
-  "version": "11.52.0-alpha.6",
+  "version": "11.52.0",
   "description": "GoodData.UI SDK - Extensions",
   "license": "LicenseRef-LICENSE",
   "author": "GoodData Corporation",
@@ -63,20 +63,20 @@
     "ts-invariant": "0.10.3",
     "tslib": "2.8.1",
     "uuid": "11.1.1",
-    "@gooddata/sdk-backend-base": "11.52.0-alpha.6",
-    "@gooddata/sdk-backend-spi": "11.52.0-alpha.6",
-    "@gooddata/sdk-embedding": "11.52.0-alpha.6",
-    "@gooddata/sdk-model": "11.52.0-alpha.6",
-    "@gooddata/sdk-ui": "11.52.0-alpha.6",
-    "@gooddata/sdk-ui-charts": "11.52.0-alpha.6",
-    "@gooddata/sdk-ui-geo": "11.52.0-alpha.6",
-    "@gooddata/sdk-ui-filters": "11.52.0-alpha.6",
-    "@gooddata/sdk-ui-semantic-search": "11.52.0-alpha.6",
-    "@gooddata/sdk-ui-kit": "11.52.0-alpha.6",
-    "@gooddata/sdk-ui-theme-provider": "11.52.0-alpha.6",
-    "@gooddata/sdk-ui-vis-commons": "11.52.0-alpha.6",
-    "@gooddata/sdk-ui-pivot": "11.52.0-alpha.6",
-    "@gooddata/util": "11.52.0-alpha.6"
+    "@gooddata/sdk-backend-base": "11.52.0",
+    "@gooddata/sdk-backend-spi": "11.52.0",
+    "@gooddata/sdk-model": "11.52.0",
+    "@gooddata/sdk-ui": "11.52.0",
+    "@gooddata/sdk-embedding": "11.52.0",
+    "@gooddata/sdk-ui-charts": "11.52.0",
+    "@gooddata/sdk-ui-filters": "11.52.0",
+    "@gooddata/sdk-ui-geo": "11.52.0",
+    "@gooddata/sdk-ui-kit": "11.52.0",
+    "@gooddata/sdk-ui-pivot": "11.52.0",
+    "@gooddata/sdk-ui-vis-commons": "11.52.0",
+    "@gooddata/sdk-ui-theme-provider": "11.52.0",
+    "@gooddata/sdk-ui-semantic-search": "11.52.0",
+    "@gooddata/util": "11.52.0"
   },
   "devDependencies": {
     "@microsoft/api-documenter": "^7.17.0",
@@ -124,12 +124,12 @@
     "typescript": "5.9.3",
     "vitest": "4.1.8",
     "vitest-dom": "0.1.1",
-    "@gooddata/eslint-config": "11.52.0-alpha.6",
-    "@gooddata/i18n-toolkit": "11.52.0-alpha.6",
-    "@gooddata/oxlint-config": "11.52.0-alpha.6",
-    "@gooddata/reference-workspace": "11.52.0-alpha.6",
-    "@gooddata/sdk-backend-mockingbird": "11.52.0-alpha.6",
-    "@gooddata/stylelint-config": "11.52.0-alpha.6"
+    "@gooddata/i18n-toolkit": "11.52.0",
+    "@gooddata/eslint-config": "11.52.0",
+    "@gooddata/oxlint-config": "11.52.0",
+    "@gooddata/reference-workspace": "11.52.0",
+    "@gooddata/sdk-backend-mockingbird": "11.52.0",
+    "@gooddata/stylelint-config": "11.52.0"
   },
   "peerDependencies": {
     "react": "^18.0.0 || ^19.0.0",