Fix 404 page and podcast page (#1766)

OperationCode · Nov 9, 2023 · 1245699 · 1245699 · vercel · Nov 9, 2023
1 parent de73d5f
commit 1245699
Show file tree

Hide file tree

Showing 15 changed files with 39 additions and 51 deletions.
diff --git a/components/Buttons/CloseButton/__tests__/__snapshots__/CloseButton.test.js.snap b/components/Buttons/CloseButton/__tests__/__snapshots__/CloseButton.test.js.snap
@@ -9,7 +9,7 @@ exports[`CloseButton should render with just required props passed 1`] = `
   type="button"
 >
   <span
-    className="ScreenReaderOnly"
+    className="sr-only"
     data-testid="SCREEN_READER_ONLY"
   >
     Close

diff --git a/components/Buttons/LinkButton/__tests__/__snapshots__/LinkButton.test.js.snap b/components/Buttons/LinkButton/__tests__/__snapshots__/LinkButton.test.js.snap
@@ -11,7 +11,7 @@ exports[`LinkButton should render with many props assigned 1`] = `
 >
   Test
   <span
-    className="ScreenReaderOnly"
+    className="sr-only"
     data-testid="SCREEN_READER_ONLY"
   >
     Opens in new window

diff --git a/components/Cards/ValueCard/__tests__/__snapshots__/ValueCard.test.js.snap b/components/Cards/ValueCard/__tests__/__snapshots__/ValueCard.test.js.snap
@@ -2,7 +2,7 @@
 
 exports[`ValueCard should render with required props 1`] = `
 <article
-  className="Card !justify-start max-w-[400px] h-[400px] p-8 overflow-y-auto"
+  className="items-center bg-white [&_svg]:fill-secondary text-secondary flex flex-col flex-nowrap justify-around m-4 min-h-[100px] min-w-[100px] p-6 shadow-md focus-visible:outline-none !justify-start max-w-[400px] h-[400px] p-8 overflow-y-auto"
 >
   <h3
     className="text-center underline"

diff --git a/components/Content/__tests__/__snapshots__/Content.test.js.snap b/components/Content/__tests__/__snapshots__/Content.test.js.snap
@@ -24,7 +24,7 @@ exports[`Content should render with many props assigned 1`] = `
             id="testing-link"
           >
             <span
-              className="ScreenReaderOnly"
+              className="sr-only"
               data-testid="SCREEN_READER_ONLY"
             >
               Scroll Link for 

diff --git a/components/ErrorDisplay/ErrorDisplay.js b/components/ErrorDisplay/ErrorDisplay.js
@@ -1,4 +1,3 @@
-import { useEffect } from 'react';
 import { number } from 'prop-types';
 import Head from 'components/head';
 
@@ -7,31 +6,22 @@ ErrorDisplay.propTypes = { statusCode: number };
 ErrorDisplay.defaultProps = { statusCode: undefined };
 
 function ErrorDisplay({ statusCode }) {
-  useEffect(() => {
-    document.body.style.overflowY = 'hidden';
-    return () => {
-      document.body.style.overflowY = '';
-    };
-  }, []);
-
   return (
     <>
       <Head title={statusCode ? `${statusCode}` : 'Error'}>
         <meta name="robots" content="noindex, nofollow" key={statusCode} />
       </Head>
 
-      <div className="bg-themeGray w-full h-full">
-        <div
-          className={
-            'h-[45vh] md:h-full text-white bg-[url("/static/images/TankFlip.gif")] flex flex-col justify-center w-full  bg-cover items-center'
-          }
-        >
-          <div className="text-center my-4 mx-auto">
-            <h1 className="text-6xl text-white">{statusCode || 'Oh no'}!</h1>
-            <p className="text-2xl text-white">
-              We&apos;re so ashamed. You definitely weren&apos;t supposed to see this...
-            </p>
-          </div>
+      <div
+        className={
+          'h-screen text-white bg-[url("/static/images/TankFlip.gif")] flex flex-col justify-center w-full  bg-cover items-center'
+        }
+      >
+        <div className="text-center my-4 mx-auto">
+          <h1 className="text-6xl text-white">{statusCode || 'Oh no'}!</h1>
+          <p className="text-2xl text-white">
+            We&apos;re so ashamed. You definitely weren&apos;t supposed to see this...
+          </p>
         </div>
       </div>
     </>

diff --git a/components/ErrorDisplay/__tests__/__snapshots__/ErrorDisplay.test.js.snap b/components/ErrorDisplay/__tests__/__snapshots__/ErrorDisplay.test.js.snap
@@ -2,26 +2,22 @@
 
 exports[`ErrorDisplay should render with just required props 1`] = `
 <div
-  className="bg-themeGray w-full h-full"
+  className="h-screen text-white bg-[url(\\"/static/images/TankFlip.gif\\")] flex flex-col justify-center w-full  bg-cover items-center"
 >
   <div
-    className="h-[45vh] md:h-full text-white bg-[url(\\"/static/images/TankFlip.gif\\")] flex flex-col justify-center w-full  bg-cover items-center"
+    className="text-center my-4 mx-auto"
   >
-    <div
-      className="text-center my-4 mx-auto"
+    <h1
+      className="text-6xl text-white"
     >
-      <h1
-        className="text-6xl text-white"
-      >
-        404
-        !
-      </h1>
-      <p
-        className="text-2xl text-white"
-      >
-        We're so ashamed. You definitely weren't supposed to see this...
-      </p>
-    </div>
+      404
+      !
+    </h1>
+    <p
+      className="text-2xl text-white"
+    >
+      We're so ashamed. You definitely weren't supposed to see this...
+    </p>
   </div>
 </div>
 `;
diff --git a/components/FeaturedJobItem/__tests__/__snapshots__/FeaturedJobItem.test.js.snap b/components/FeaturedJobItem/__tests__/__snapshots__/FeaturedJobItem.test.js.snap
@@ -15,7 +15,7 @@ exports[`FeaturedJobItem should render with many props assigned 1`] = `
       Experienced React Engineer
     </h6>
     <span
-      className="ScreenReaderOnly"
+      className="sr-only"
       data-testid="SCREEN_READER_ONLY"
     >
       Opens in new window
@@ -99,7 +99,7 @@ exports[`FeaturedJobItem should render with required props 1`] = `
       DevOps Engineer
     </h6>
     <span
-      className="ScreenReaderOnly"
+      className="sr-only"
       data-testid="SCREEN_READER_ONLY"
     >
       Opens in new window

diff --git a/components/Forms/RegistrationForm/__tests__/__snapshots__/RegistrationForm.test.js.snap b/components/Forms/RegistrationForm/__tests__/__snapshots__/RegistrationForm.test.js.snap
@@ -247,7 +247,7 @@ exports[`RegistrationForm should render with required props 1`] = `
           >
             Operation Code's Code of Conduct.
             <span
-              className="ScreenReaderOnly"
+              className="sr-only"
               data-testid="SCREEN_READER_ONLY"
             >
               Opens in new window

diff --git a/components/Heading/__tests__/__snapshots__/Heading.test.js.snap b/components/Heading/__tests__/__snapshots__/Heading.test.js.snap
@@ -29,7 +29,7 @@ exports[`Heading should render with required props 1`] = `
         id="test-link"
       >
         <span
-          className="ScreenReaderOnly"
+          className="sr-only"
           data-testid="SCREEN_READER_ONLY"
         >
           Scroll Link for 

diff --git a/components/Modal/__tests__/__snapshots__/Modal.test.js.snap b/components/Modal/__tests__/__snapshots__/Modal.test.js.snap
@@ -29,7 +29,7 @@ exports[`Modal should render with many props assigned 1`] = `
       type="button"
     >
       <span
-        class="ScreenReaderOnly"
+        class="sr-only"
         data-testid="SCREEN_READER_ONLY"
       >
         Close

diff --git a/components/OutboundLink/__tests__/__snapshots__/OutboundLink.test.js.snap b/components/OutboundLink/__tests__/__snapshots__/OutboundLink.test.js.snap
@@ -10,7 +10,7 @@ exports[`OutboundLink should render with required props 1`] = `
 >
   Test
   <span
-    className="ScreenReaderOnly"
+    className="sr-only"
     data-testid="SCREEN_READER_ONLY"
   >
     Opens in new window

diff --git a/components/ReusableSections/JoinSection/__tests__/__snapshots__/JoinSection.test.js.snap b/components/ReusableSections/JoinSection/__tests__/__snapshots__/JoinSection.test.js.snap
@@ -24,7 +24,7 @@ exports[`JoinSection should render 1`] = `
             id="join-our-thriving-community-link"
           >
             <span
-              className="ScreenReaderOnly"
+              className="sr-only"
               data-testid="SCREEN_READER_ONLY"
             >
               Scroll Link for 
@@ -61,7 +61,7 @@ exports[`JoinSection should render 1`] = `
     >
       Never heard of Slack before?
       <span
-        className="ScreenReaderOnly"
+        className="sr-only"
         data-testid="SCREEN_READER_ONLY"
       >
         Opens in new window

diff --git a/components/SocialMedia/SocialMediaItem/__tests__/__snapshots__/SocialMediaItem.test.js.snap b/components/SocialMedia/SocialMediaItem/__tests__/__snapshots__/SocialMediaItem.test.js.snap
@@ -12,15 +12,15 @@ exports[`SocialMediaItem should render with required props 1`] = `
     target="_blank"
   >
     <span
-      className="ScreenReaderOnly"
+      className="sr-only"
       data-testid="SCREEN_READER_ONLY"
     >
       Operation Code's 
       Facebook
     </span>
     <svg />
     <span
-      className="ScreenReaderOnly"
+      className="sr-only"
       data-testid="SCREEN_READER_ONLY"
     >
       Opens in new window

diff --git a/next.config.js b/next.config.js
@@ -36,6 +36,8 @@ const nextConfig = {
       s3hostName,
       'user-images.githubusercontent.com',
       'ssl-static.libsyn.com',
+      'static.libsyn.com',
+      'libsyn.com',
       'i.ytimg.com',
     ],
   },

diff --git a/pages/podcast.js b/pages/podcast.js
@@ -44,7 +44,7 @@ export async function getStaticProps() {
     return { props: { episodes }, revalidate: ONE_DAY };
   }
 
-  // Request failed or RSS Feed is broken
+  // Request failed or RSS Feed is broken... Break the build!
   throw new Error('getStaticProps in /podcast failed.');
 }