Remove Loading Component added on 1753

gnosis · Nov 5, 2021 · 37f8dea · 37f8dea
1 parent 1ba6f4c
commit 37f8dea
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 8 deletions.
diff --git a/src/custom/hooks/useFetchProfile.ts b/src/custom/hooks/useFetchProfile.ts
@@ -5,13 +5,13 @@ import { ProfileData } from 'api/gnosisProtocol/api'
 
 type FetchProfileState = {
   profileData: ProfileData | null
-  //error: string
+  error: string
   isLoading: boolean
 }
 
 const emptyState: FetchProfileState = {
   profileData: null,
-  //error: '',
+  error: '',
   isLoading: false,
 }
 
@@ -22,6 +22,7 @@ export default function useFetchProfile(): FetchProfileState {
   const [profile, setProfile] = useState<FetchProfileState>(emptyState)
 
   useEffect(() => {
+    setProfile({ ...emptyState, isLoading: true })
     async function fetchAndSetProfileData() {
       try {
         if (chainId && account) {

diff --git a/src/custom/pages/Profile/index.tsx b/src/custom/pages/Profile/index.tsx
@@ -26,9 +26,26 @@ import { SupportedChainId as ChainId } from 'constants/chains'
 export default function Profile() {
   const referralLink = useReferralLink()
   const { account, chainId } = useActiveWeb3React()
-  const { profileData, isLoading } = useFetchProfile()
+  const { profileData, isLoading, error } = useFetchProfile()
   const lastUpdated = useTimeAgo(profileData?.lastUpdated)
 
+  const renderNotificationMessages = () => {
+    return (
+      <>
+        {error && (
+          <StyledNotificationBanner isVisible level="error" canClose={false}>
+            There was an error loading your profile data. Please try again later.
+          </StyledNotificationBanner>
+        )}
+        {chainId && chainId !== ChainId.MAINNET && (
+          <StyledNotificationBanner isVisible level="info" canClose={false}>
+            Profile data is only available for mainnet. Please change the network to see it.
+          </StyledNotificationBanner>
+        )}
+      </>
+    )
+  }
+
   return (
     <Wrapper>
       <GridWrap>
@@ -57,11 +74,7 @@ export default function Profile() {
             </Loader>
           )}
         </CardHead>
-        {chainId && chainId !== ChainId.MAINNET && (
-          <StyledNotificationBanner isVisible level="info" canClose={false}>
-            Profile data is only available for mainnet. Please change the network to see it.
-          </StyledNotificationBanner>
-        )}
+        {renderNotificationMessages()}
         <ChildWrapper>
           <Txt fs={16}>
             <strong>Your referral url</strong>