@rh-support/troubleshoot 0.2.58 → 0.2.59

package/lib/esm/components/CaseEditView/Tabs/CaseDiscussion/PostComment.js

@@ -276,7 +276,7 @@ export function PostComment(props) {
                 onClipboardPaste,
             } }),
         isProcessing ? (React.createElement(LoadingIndicator, { isInline: true, size: "sm" })) : (React.createElement("div", { className: "pf-l-flex post-comment-btn-group push-top-narrow" },
-            isModalOpen && loggedInUser.data.isInternal && isPublic && (React.createElement(VerifyCaseStatusModal, { onClose: onCancel, onConfirm: onConfirm, isUpdating: isPostingComment, caseNumber: props.caseNumber })),
+            isModalOpen && loggedInUserRights.data.isInternal() && isPublic && (React.createElement(VerifyCaseStatusModal, { onClose: onCancel, onConfirm: onConfirm, isUpdating: isPostingComment, caseNumber: props.caseNumber })),
             React.createElement(Button, { "data-tracking-id": "postcomment-submit-button", type: "button", variant: ButtonVariant.primary, onClick: loggedInUser.data.isInternal && isPublic ? onBtnClick : isNotInternalConfirm, isDisabled: isPostCommentDisabled, className: "postcomment-submit" },
                 React.createElement(Trans, null, "Submit")),
             canAddAttachments &&

package/lib/esm/components/CaseEditView/Tabs/CaseDiscussion/VerifyCaseStatusModal/VerifyCaseStatusModal.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"VerifyCaseStatusModal.d.ts","sourceRoot":"","sources":["../../../../../../../src/components/CaseEditView/Tabs/CaseDiscussion/VerifyCaseStatusModal/VerifyCaseStatusModal.tsx"],"names":[],"mappings":"AAiBA,UAAU,MAAM;IACZ,SAAS,EAAE,CAAC,WAAW,CAAC,EAAE,MAAM,KAAK,IAAI,CAAC;IAC1C,OAAO,EAAE,MAAM,IAAI,CAAC;IACpB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,UAAU,EAAE,OAAO,CAAC;CACvB;AAED,wBAAgB,qBAAqB,CAAC,KAAK,EAAE,MAAM,eAiIlD"}
+{"version":3,"file":"VerifyCaseStatusModal.d.ts","sourceRoot":"","sources":["../../../../../../../src/components/CaseEditView/Tabs/CaseDiscussion/VerifyCaseStatusModal/VerifyCaseStatusModal.tsx"],"names":[],"mappings":"AAiBA,UAAU,MAAM;IACZ,SAAS,EAAE,CAAC,WAAW,CAAC,EAAE,MAAM,KAAK,IAAI,CAAC;IAC1C,OAAO,EAAE,MAAM,IAAI,CAAC;IACpB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,UAAU,EAAE,OAAO,CAAC;CACvB;AAED,wBAAgB,qBAAqB,CAAC,KAAK,EAAE,MAAM,eAqIlD"}

package/lib/esm/components/CaseEditView/Tabs/CaseDiscussion/VerifyCaseStatusModal/VerifyCaseStatusModal.js

@@ -53,24 +53,29 @@ export function VerifyCaseStatusModal(props) {
     const { isCaseUpdating } = caseState;
     const canEditCase = useCanEditCase();
     const caseDispatch = useCaseDispatch();
-    const [statusChange, setStatusChange] = useState();
+    const [statusChange, setStatusChange] = useState(status);
     const [isVerifyCaseStatusLoading, setIsVerifyCaseStatusLoading] = useState(false);
     const onSubmit = () => __awaiter(this, void 0, void 0, function* () {
+        yield props.onConfirm();
         yield updateStatusState(statusChange);
         setIsVerifyCaseStatusLoading(true);
-        props.onConfirm();
     });
     const updateStatusState = (statusVal, onStatusUpdate = null) => __awaiter(this, void 0, void 0, function* () {
-        setStatusUpdating(caseOverviewDispatch, true);
-        try {
-            yield updateCaseDetails(caseDispatch, props.caseNumber, { status: statusVal });
-            setStatusUpdating(caseOverviewDispatch, false);
-            onStatusUpdate && (yield onStatusUpdate());
-            ToastNotification.addSuccessMessage(t('Status has been successfully updated'));
+        if (status !== statusVal) {
+            setStatusUpdating(caseOverviewDispatch, true);
+            try {
+                yield updateCaseDetails(caseDispatch, props.caseNumber, { status: statusVal });
+                setStatusUpdating(caseOverviewDispatch, false);
+                onStatusUpdate && (yield onStatusUpdate());
+                ToastNotification.addSuccessMessage(t('Status has been successfully updated'));
+            }
+            catch (e) {
+                setStatusUpdating(caseOverviewDispatch, false);
+                ToastNotification.addDangerMessage(t('Status failed to update'));
+            }
         }
-        catch (e) {
-            setStatusUpdating(caseOverviewDispatch, false);
-            ToastNotification.addDangerMessage(t('Status failed to update'));
+        else {
+            return;
         }
     });
     const onStatusChange = (event, newStatus) => __awaiter(this, void 0, void 0, function* () {
@@ -90,6 +95,6 @@ export function VerifyCaseStatusModal(props) {
             React.createElement("span", { className: "form-required", "aria-hidden": true }, "*")),
         React.createElement(Select, { "aria-label": t(status), id: "verify-case-status", selections: statusChange, onSelect: onStatusChange, isOpen: showDropdown, isDisabled: !canManageCase ||
                 caseOverviewState.allCaseStatuses.isFetching ||
-                (caseOverviewState.caseStatusUpdating && isCaseUpdating), menuAppendTo: document.body, validated: !status ? 'error' : 'default', placeholderText: t(status), onToggle: onDropdownToggle }, caseOverviewState.allCaseStatuses.data.map((option, index) => (React.createElement(SelectOption, { key: index, value: option },
+                (caseOverviewState.caseStatusUpdating && isCaseUpdating), menuAppendTo: document.body, validated: !status ? 'error' : 'default', placeholderText: t(status), onToggle: onDropdownToggle }, caseOverviewState.allCaseStatuses.data.map((option, index) => (React.createElement(SelectOption, { key: index, value: option, isDisabled: option === status },
             React.createElement(Trans, null, option)))))));
 }

package/lib/esm/components/Recommendations/ClusterRecommendationToast.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ClusterRecommendationToast.d.ts","sourceRoot":"","sources":["../../../../src/components/Recommendations/ClusterRecommendationToast.tsx"],"names":[],"mappings":"AAIA,OAAc,EAAE,EAAE,EAAmC,MAAM,OAAO,CAAC;AAUnE,UAAU,MAAM;CAAG;AAEnB,QAAA,MAAM,0BAA0B,EAAE,EAAE,CAAC,MAAM,CAmD1C,CAAC;AAEF,eAAe,0BAA0B,CAAC"}
+{"version":3,"file":"ClusterRecommendationToast.d.ts","sourceRoot":"","sources":["../../../../src/components/Recommendations/ClusterRecommendationToast.tsx"],"names":[],"mappings":"AAIA,OAAc,EAAE,EAAE,EAAmC,MAAM,OAAO,CAAC;AAUnE,UAAU,MAAM;CAAG;AAEnB,QAAA,MAAM,0BAA0B,EAAE,EAAE,CAAC,MAAM,CAwD1C,CAAC;AAEF,eAAe,0BAA0B,CAAC"}

package/lib/esm/components/Recommendations/ClusterRecommendationToast.js

@@ -26,7 +26,9 @@ const ClusterRecommendationToast = () => {
     };
     // To prevent toast to reopen even after user close it we need to check cluster id
     useEffect(() => {
-        if (!isEmpty(openshiftClusterID) && previousOpenShiftClusterId !== openshiftClusterID)
+        if (!isEmpty(openshiftClusterID) &&
+            previousOpenShiftClusterId !== openshiftClusterID &&
+            clusterRecommendations.data.length)
             setToastVisibility(true);
         // eslint-disable-next-line react-hooks/exhaustive-deps
     }, [clusterRecommendations.data]);

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@rh-support/troubleshoot",
-    "version": "0.2.58",
+    "version": "0.2.59",
     "publishConfig": {
         "access": "public",
         "registry": "https://registry.npmjs.org"
@@ -143,5 +143,5 @@
         "not ie <= 11",
         "not op_mini all"
     ],
-    "gitHead": "8ce3e080eafb6dd8df726bed5b7f2d54acf8cc22"
+    "gitHead": "053a5aa02f56e75747823ec36596180ed6427ab7"
 }