Merge branch 'main' of https://github.com/kyma-project/busola into post-conversations

Author: chriskari

.github/workflows/accessibility-tests.yml

@@ -7,8 +7,13 @@ on:
     types: [opened, edited, synchronize, reopened, ready_for_review]
     paths:
       - '.github/workflows/accessibility-tests.yml'
-      - 'tests/integration/tests/accessibility/**'
-      - 'tests/integration/support/**'
+      - 'resources/**'
+      - 'tests/integration/**'
+      - 'nginx/**'
+      - 'src/**'
+      - 'backend/**'
+      - 'kyma/**'
+      - 'Dockerfile*'
 
 jobs:
   run-accessibility-tests:
@@ -36,11 +41,12 @@ jobs:
         shell: bash
         env:
           ACC_AMP_TOKEN: ${{ secrets.ACC_AMP_TOKEN }}
+          IS_PR: ${{ github.event_name == 'pull_request' || github.event_name == 'pull_request_target' }}
         run: |
           k3d kubeconfig get kyma > tests/integration/fixtures/kubeconfig.yaml
           export CYPRESS_DOMAIN=http://localhost:3001          
           cd tests/integration
-          npm ci && ACC_AMP_TOKEN=$ACC_AMP_TOKEN npm run "test:accesibility"
+          npm ci && ACC_AMP_TOKEN=$ACC_AMP_TOKEN IS_PR=$IS_PR npm run "test:accesibility"
       - name: Uploads artifacts
         uses: actions/upload-artifact@v4
         if: ${{ always() }}

kyma/environments/dev/config.yaml

@@ -100,4 +100,4 @@ config:
     COLUMN_LAYOUT:
       isEnabled: true
     SNOW:
-      isEnabled: true
+      isEnabled: false

kyma/environments/stage/config.yaml

@@ -99,4 +99,4 @@ config:
     COLUMN_LAYOUT:
       isEnabled: true
     SNOW:
-      isEnabled: true
+      isEnabled: false

public/defaultConfig.yaml

@@ -80,4 +80,4 @@ config:
       isEnabled: true
       link: https://sapinsights.eu.qualtrics.com/jfe/form/SV_d3UPNymSgUHAb9Y?product=SAP%20BTP,%20Kyma%20Runtime&product_filter=Kyma
     SNOW:
-      isEnabled: true
+      isEnabled: false

src/components/BusolaExtensions/ExtensibilityStarterForm.scss

@@ -6,6 +6,14 @@
   ui5-checkbox {
     margin-top: -6.75px;
   }
+
+  @media (max-width: 1200px) {
+    grid-template-columns: 2fr 8fr 8fr 3fr;
+  }
+
+  @media (max-width: 480px) {
+    grid-template-columns: 1fr;
+  }
 }
 
 .resource-form--unset-height {

src/hooks/useSentry.js

@@ -1,30 +1,37 @@
-import { useState } from 'react';
+import { useState, useEffect } from 'react';
 import * as Sentry from '@sentry/react';
 import { useFeature } from './useFeature';
 
-export function useSentry() {
-  const [dsn, setDsn] = useState();
-
-  const initSentry = dsn => {
-    Sentry.init({
-      dsn,
-      release: 'busola',
-      integrations: [new Sentry.browserTracingIntegration()],
-      tracesSampleRate: 1.0,
-    });
-  };
+const initSentry = dsn => {
+  Sentry.init({
+    dsn,
+    release: 'busola',
+    integrations: [
+      new Sentry.browserTracingIntegration(),
+      new Sentry.replayIntegration(),
+    ],
+    tracesSampleRate: 1.0,
+    // Session Replay
+    replaysSessionSampleRate: 0.1,
+    replaysOnErrorSampleRate: 1.0,
+  });
+};
 
-  try {
-    const feature = useFeature('SENTRY') || {};
-    if (feature.isEnabled && feature.config?.dsn) {
-      const nextDsn = feature.config.dsn;
+export function useSentry() {
+  const [dsn, setDsn] = useState(null);
+  const feature = useFeature('SENTRY');
 
-      if (nextDsn !== dsn) {
-        setDsn(nextDsn);
-        initSentry(nextDsn);
+  useEffect(() => {
+    try {
+      if (feature?.isEnabled && feature?.config?.dsn) {
+        const nextDsn = feature.config.dsn;
+        if (nextDsn !== dsn) {
+          setDsn(nextDsn);
+          initSentry(nextDsn);
+        }
       }
+    } catch (e) {
+      console.warn('Sentry not enabled due to error', e);
     }
-  } catch (e) {
-    console.warn('Sentry not enabled due to error', e);
-  }
+  }, [feature, dsn]);
 }

src/resources/Namespaces/YamlUpload/YamlResourcesList.js

@@ -102,7 +102,7 @@ export function YamlResourcesList({ resourcesData }) {
                   <span style={{ fontWeight: 'bold' }}></span>
                 </Trans>
               </p>
-              <Title level="H4" size="H4" className="sap-margin-top-small">
+              <Title level="H2" size="H4" className="sap-margin-top-small">
                 {t('upload-yaml.uploaded-resources')}
               </Title>
               <SeparatorLine style={{ margin: '0rem -1rem' }} />

src/resources/ReplicaSets/ReplicaSetDetails.js

@@ -120,11 +120,19 @@ export function ReplicaSetsDetails(props) {
       value: resource => <div>{resource?.status?.replicas ?? 0}</div>,
     },
   ];
-
   const statusConditions = resource => {
     return resource?.status?.conditions?.map(condition => {
+      const overridenStatus = () => {
+        if (condition.type === 'ReplicaFailure')
+          return condition.status === 'True' ? 'Negative' : 'Positive';
+        return undefined;
+      };
       return {
-        header: { titleText: condition.type, status: condition.status },
+        header: {
+          titleText: condition.type,
+          status: condition.status,
+          overrideStatusType: overridenStatus(),
+        },
         message: condition.message,
       };
     });

src/shared/components/DynamicPageComponent/DynamicPageComponent.scss

@@ -14,6 +14,7 @@
 
   .bold-title {
     font-family: var(--sapObjectHeader_Title_FontFamily);
+    text-wrap-mode: wrap;
   }
 
   .description {

src/shared/components/MonacoEditorESM/Editor.js

@@ -105,6 +105,7 @@ export function Editor({
             design="Negative"
             hideCloseButton
             className="break-word sap-margin-small"
+            role="alert"
           >
             {t('common.create-form.editor-error', { error })}
           </MessageStrip>
@@ -113,6 +114,7 @@ export function Editor({
           <MessageStrip
             className="break-word sap-margin-small"
             design="Critical"
+            role="alert"
           >
             {t('common.create-form.autocomplete-unavailable-error', {
               error: schemaError.error || schemaError.message || schemaError,
@@ -124,6 +126,7 @@ export function Editor({
             design="Critical"
             hideCloseButton
             className="break-word sap-margin-small"
+            role="alert"
           >
             {warnings.map(m => (
               <span