From ded78c66399b473843e771dacff096d0b1b8059b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?V=C3=ADctor=20Losada=20Hern=C3=A1ndez?= Date: Mon, 6 May 2024 23:32:18 +0200 Subject: [PATCH] better error handling in jsx --- .../pages/archivePage/archivePage.jsx | 22 +++++++++++-------- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/client/homebrew/pages/archivePage/archivePage.jsx b/client/homebrew/pages/archivePage/archivePage.jsx index 90d9104a0..4dc921fd3 100644 --- a/client/homebrew/pages/archivePage/archivePage.jsx +++ b/client/homebrew/pages/archivePage/archivePage.jsx @@ -263,24 +263,19 @@ const ArchivePage = createClass({ if(title === '') {return (

Whenever you want, just start typing...

);} - if (!brewCollection || brewCollection.length === 0) { - return ( -
-

No brews found

-
- ); - }; + if (error) { console.log('render Error: ', error); let errorMessage; switch (error.errorCode) { case '404': - errorMessage = "We didn't find any brew"; + errorMessage = "404 - We didn't find any brew"; break; case '503': - errorMessage = 'Your search is not specific enough. Too many brews meet this criteria for us to display them.'; + errorMessage = ' 503 - Your search is not specific enough. Too many brews meet this criteria for us to display them.'; break; case '500': + errorMessage = "500 - We don't know what happened." default: errorMessage = 'An unexpected error occurred'; } @@ -291,6 +286,15 @@ const ArchivePage = createClass({ ); }; + + if (!brewCollection || brewCollection.length === 0) { + return ( +
+

No brews found

+
+ ); + }; +