[ui] make reload button on RepositoryLocationErrorDialog optional

dagster-io · Dec 4, 2024 · 4e4ce93 · 4e4ce93
1 parent 396a8af
commit 4e4ce93
Showing 1 changed file with 11 additions and 7 deletions.
diff --git a/js_modules/dagster-ui/packages/ui-core/src/workspace/RepositoryLocationErrorDialog.tsx b/js_modules/dagster-ui/packages/ui-core/src/workspace/RepositoryLocationErrorDialog.tsx
@@ -9,7 +9,7 @@ interface Props {
   error: PythonErrorFragment | {message: string} | null;
   reloading: boolean;
   onDismiss: () => void;
-  onTryReload: () => void;
+  onTryReload?: () => void;
 }
 
 export const RepositoryLocationErrorDialog = (props: Props) => {
@@ -27,9 +27,11 @@ export const RepositoryLocationErrorDialog = (props: Props) => {
         <ErrorContents location={location} error={error} />
       </DialogBody>
       <DialogFooter>
-        <Button onClick={onTryReload} loading={reloading} intent="primary">
-          Reload again
-        </Button>
+        {onTryReload && (
+          <Button onClick={onTryReload} loading={reloading} intent="primary">
+            Reload again
+          </Button>
+        )}
         <Button onClick={onDismiss}>Dismiss</Button>
       </DialogFooter>
     </Dialog>
@@ -50,9 +52,11 @@ export const RepositoryLocationNonBlockingErrorDialog = (props: Props) => {
         <ErrorContents location={location} error={error} />
       </DialogBody>
       <DialogFooter>
-        <Button onClick={onTryReload} loading={reloading} intent="primary">
-          Reload
-        </Button>
+        {onTryReload && (
+          <Button onClick={onTryReload} loading={reloading} intent="primary">
+            Reload
+          </Button>
+        )}
         <Button onClick={onDismiss}>Close</Button>
       </DialogFooter>
     </Dialog>