From 996bdfebdbcf7449474ba1b61c24fb9e211d698b Mon Sep 17 00:00:00 2001
From: sergeyteleshev <iamsergeyteleshev@gmail.com>
Date: Thu, 18 Apr 2024 18:25:59 +0200
Subject: [PATCH] CB-4268 show error for all auth tabs

---
 .../src/Dialog/AuthDialog.tsx                 | 49 +++++++++----------
 1 file changed, 23 insertions(+), 26 deletions(-)

diff --git a/webapp/packages/plugin-authentication/src/Dialog/AuthDialog.tsx b/webapp/packages/plugin-authentication/src/Dialog/AuthDialog.tsx
index 2cbfaaadd3..54e5e0b4bb 100644
--- a/webapp/packages/plugin-authentication/src/Dialog/AuthDialog.tsx
+++ b/webapp/packages/plugin-authentication/src/Dialog/AuthDialog.tsx
@@ -236,34 +236,31 @@ export const AuthDialog: DialogComponent<IAuthOptions, null> = observer(function
             </Form>
           )}
         </CommonDialogBody>
-        {/* TODO should we always display footer for both: local and federated? */}
-        {(!federate || (federate && state.isTooManySessions)) && (
-          <CommonDialogFooter>
-            <Container>
-              {state.isTooManySessions && (
-                <Checkbox
-                  className={s(styles, { tooManySessionsCheckbox: true })}
-                  checked={state.isForceSessionsLogout}
-                  name="forceSessionLogout"
-                  label={translate('authentication_auth_force_session_logout')}
-                  onClick={e => {
-                    state.setForceSessionsLogout(e.currentTarget.checked);
-                  }}
+        <CommonDialogFooter>
+          <Container>
+            {state.isTooManySessions && (
+              <Checkbox
+                className={s(styles, { tooManySessionsCheckbox: true })}
+                checked={state.isForceSessionsLogout}
+                name="forceSessionLogout"
+                label={translate('authentication_auth_force_session_logout')}
+                onClick={e => {
+                  state.setForceSessionsLogout(e.currentTarget.checked);
+                }}
+              />
+            )}
+            <AuthDialogFooter authAvailable={!dialogData.configure} isAuthenticating={dialogData.authenticating} onLogin={() => login(linkUser)}>
+              {errorDetails.name && (
+                <ErrorMessage
+                  className={s(styles, { errorMessage: true })}
+                  text={errorDetails.message || errorDetails.name}
+                  hasDetails={errorDetails.hasDetails}
+                  onShowDetails={errorDetails.open}
                 />
               )}
-              <AuthDialogFooter authAvailable={!dialogData.configure} isAuthenticating={dialogData.authenticating} onLogin={() => login(linkUser)}>
-                {errorDetails.name && (
-                  <ErrorMessage
-                    className={s(styles, { errorMessage: true })}
-                    text={errorDetails.message || errorDetails.name}
-                    hasDetails={errorDetails.hasDetails}
-                    onShowDetails={errorDetails.open}
-                  />
-                )}
-              </AuthDialogFooter>
-            </Container>
-          </CommonDialogFooter>
-        )}
+            </AuthDialogFooter>
+          </Container>
+        </CommonDialogFooter>
       </CommonDialogWrapper>
     </TabsState>
   );