From 35bee5ca6869cc29c0753579b531af53e360ac4e Mon Sep 17 00:00:00 2001
From: EhEhRon91 <65462273+EhEhRon91@users.noreply.github.com>
Date: Mon, 20 Sep 2021 05:57:05 -0400
Subject: [PATCH] fix: message removed when not logged in (#43411)

* fix: message removed when not logged in

* removed unnecessary code from DonateCompletion.tsx that was added

* now using isSignedIn in props and not donationFromState
---
 client/src/components/Donation/DonateCompletion.tsx | 4 +++-
 client/src/components/Donation/DonateForm.tsx       | 6 +++++-
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/client/src/components/Donation/DonateCompletion.tsx b/client/src/components/Donation/DonateCompletion.tsx
index d2b405ae59..955bbdb6e3 100644
--- a/client/src/components/Donation/DonateCompletion.tsx
+++ b/client/src/components/Donation/DonateCompletion.tsx
@@ -11,6 +11,7 @@ type DonateCompletionProps = {
   redirecting: boolean;
   reset: () => unknown;
   success: boolean;
+  isSignedIn: boolean;
 };
 
 function DonateCompletion({
@@ -18,6 +19,7 @@ function DonateCompletion({
   reset,
   success,
   redirecting,
+  isSignedIn,
   error = null
 }: DonateCompletionProps): JSX.Element {
   /* eslint-disable no-nested-ternary */
@@ -50,7 +52,7 @@ function DonateCompletion({
         {success && (
           <div>
             <p>{t('donate.free-tech')}</p>
-            <p>{t('donate.no-halo')}</p>
+            {isSignedIn && <p>{t('donate.no-halo')}</p>}
           </div>
         )}
         {error && <p>{error}</p>}
diff --git a/client/src/components/Donation/DonateForm.tsx b/client/src/components/Donation/DonateForm.tsx
index e4c862f710..e6fe05569b 100644
--- a/client/src/components/Donation/DonateForm.tsx
+++ b/client/src/components/Donation/DonateForm.tsx
@@ -277,6 +277,7 @@ class DonateForm extends Component<DonateFormProps, DonateFormComponentState> {
     redirecting: boolean;
     success: boolean;
     error: string | null;
+    isSignedIn: boolean;
     reset: () => unknown;
   }) {
     return <DonateCompletion {...props} />;
@@ -359,7 +360,8 @@ class DonateForm extends Component<DonateFormProps, DonateFormComponentState> {
   render() {
     const {
       donationFormState: { processing, success, error, redirecting },
-      isMinimalForm
+      isMinimalForm,
+      isSignedIn
     } = this.props;
 
     if (success || error) {
@@ -368,6 +370,7 @@ class DonateForm extends Component<DonateFormProps, DonateFormComponentState> {
         redirecting,
         success,
         error,
+        isSignedIn,
         reset: this.resetDonation
       });
     }
@@ -381,6 +384,7 @@ class DonateForm extends Component<DonateFormProps, DonateFormComponentState> {
             redirecting,
             success,
             error,
+            isSignedIn,
             reset: this.resetDonation
           })}
         <div className={processing || redirecting ? 'hide' : ''}>