TMP trigger actions

Author: cadivus

packages/authenticator/amplify_authenticator/lib/src/blocs/auth/auth_bloc.dart

@@ -377,6 +377,8 @@ class StateMachineBloc
         final result = await _authService.signIn(data.username, data.password);
         await _processSignInResult(result, isSocialSignIn: false);
       } else if (data is AuthSocialSignInData) {
+        // Do not await a social sign-in since multiple sign-in attempts
+        // can occur.
         await _authService
             .signInWithProvider(
               data.provider,
@@ -390,13 +392,6 @@ class StateMachineBloc
                   ? logger.info
                   : logger.error;
               log('Error signing in', error, stackTrace);
-              // Emit exception so that the UI can exit loading state
-              _exceptionController.add(
-                AuthenticatorException(
-                  error,
-                  showBanner: error is! UserCancelledException,
-                ),
-              );
             });
       } else {
         throw StateError('Bad sign in data: $data');

packages/authenticator/amplify_authenticator/lib/src/state/authenticator_state.dart

@@ -529,15 +529,10 @@ class AuthenticatorState extends ChangeNotifier {
     _setIsBusy(false);
   }
 
-  /// Perform social sign in with the given provider
+  /// Perform sicial sign in with the given provider
   Future<void> signInWithProvider(AuthProvider provider) async {
-    _setIsBusy(true);
     final signInData = AuthSocialSignInData(provider: provider);
     _authBloc.add(AuthSignIn(signInData));
-    // Wait for either a state change (sign-in success/failure) or an exception
-    // (including user cancellation)
-    await nextBlocEvent();
-    _setIsBusy(false);
   }
 
   /// Sign out the currecnt user

packages/authenticator/amplify_authenticator/lib/src/widgets/social/social_button.dart

@@ -193,53 +193,47 @@ class _SocialSignInButtonState
   @override
   Widget build(BuildContext context) {
     final resolver = stringResolver.buttons;
-    final isBusy = state.isBusy;
-    return Opacity(
-      opacity: isBusy ? 0.5 : 1.0,
-      child: ConstrainedBox(
-        constraints: const BoxConstraints(minHeight: 40),
-        child: OutlinedButton(
-          focusNode: focusNode,
-          style: ButtonStyle(
-            foregroundColor: getButtonForegroundColor(context),
-          ),
-          onPressed: isBusy
-              ? null
-              : () => state.signInWithProvider(widget.provider),
-          child: LayoutBuilder(
-            builder: (context, constraints) {
-              final padding = calculatePadding(constraints);
-              return Padding(
-                padding: EdgeInsets.symmetric(horizontal: padding),
-                child: Row(
-                  mainAxisAlignment: padding == 0
-                      ? MainAxisAlignment.center
-                      : MainAxisAlignment.start,
-                  children: [
-                    SizedBox.square(
-                      dimension: logoSize,
-                      child: Padding(
-                        padding: widget.provider.logoInsets,
-                        child: icon,
-                      ),
+    return ConstrainedBox(
+      constraints: const BoxConstraints(minHeight: 40),
+      child: OutlinedButton(
+        focusNode: focusNode,
+        style: ButtonStyle(foregroundColor: getButtonForegroundColor(context)),
+        onPressed: state.isBusy
+            ? null
+            : () => state.signInWithProvider(widget.provider),
+        child: LayoutBuilder(
+          builder: (context, constraints) {
+            final padding = calculatePadding(constraints);
+            return Padding(
+              padding: EdgeInsets.symmetric(horizontal: padding),
+              child: Row(
+                mainAxisAlignment: padding == 0
+                    ? MainAxisAlignment.center
+                    : MainAxisAlignment.start,
+                children: [
+                  SizedBox.square(
+                    dimension: logoSize,
+                    child: Padding(
+                      padding: widget.provider.logoInsets,
+                      child: icon,
                     ),
-                    const SizedBox(width: spacing),
-                    Expanded(
-                      child: Padding(
-                        padding: const EdgeInsets.symmetric(vertical: 4),
-                        child: Text(
-                          resolver.resolve(
-                            context,
-                            ButtonResolverKey.signInWith(widget.provider),
-                          ),
+                  ),
+                  const SizedBox(width: spacing),
+                  Expanded(
+                    child: Padding(
+                      padding: const EdgeInsets.symmetric(vertical: 4),
+                      child: Text(
+                        resolver.resolve(
+                          context,
+                          ButtonResolverKey.signInWith(widget.provider),
                         ),
                       ),
                     ),
-                  ],
-                ),
-              );
-            },
-          ),
+                  ),
+                ],
+              ),
+            );
+          },
         ),
       ),
     );