fix: Unlinked licenses handling

Author: wzrdx

app/license/[licenseType]/[licenseId]/page.tsx

@@ -10,49 +10,49 @@ import * as types from '@/typedefs/blockchain';
 import { Skeleton } from '@heroui/skeleton';
 import { cache, Suspense } from 'react';
 
-export async function generateMetadata({ params }) {
-    const { licenseType, licenseId } = await params;
-    const errorMetadata = {
-        title: 'Error',
-        openGraph: {
-            title: 'Error',
-        },
-    };
-
-    if (!licenseType || !['ND', 'MND', 'GND'].includes(licenseType)) {
-        return errorMetadata;
-    }
+export async function generateMetadata({ params }) {
+    const { licenseType, licenseId } = await params;
+    const errorMetadata = {
+        title: 'Error',
+        openGraph: {
+            title: 'Error',
+        },
+    };
+
+    if (!licenseType || !['ND', 'MND', 'GND'].includes(licenseType)) {
+        return errorMetadata;
+    }
 
     const licenseIdNum = parseInt(licenseId);
 
-    if (isNaN(licenseIdNum) || licenseIdNum < 0 || licenseIdNum > 10000) {
-        return errorMetadata;
-    }
-
-    const canonical = `/license/${encodeURIComponent(licenseType)}/${encodeURIComponent(licenseId)}`;
-    const errorMetadataWithCanonical = {
-        ...errorMetadata,
-        alternates: {
-            canonical,
-        },
-    };
-
-    try {
-        await fetchLicense(licenseType, licenseId, config.environment);
-    } catch (error) {
-        return errorMetadataWithCanonical;
-    }
-
-    return {
-        title: `License #${licenseId}`,
-        openGraph: {
-            title: `License #${licenseId}`,
-        },
-        alternates: {
-            canonical,
-        },
-    };
-}
+    if (isNaN(licenseIdNum) || licenseIdNum < 0 || licenseIdNum > 10000) {
+        return errorMetadata;
+    }
+
+    const canonical = `/license/${encodeURIComponent(licenseType)}/${encodeURIComponent(licenseId)}`;
+    const errorMetadataWithCanonical = {
+        ...errorMetadata,
+        alternates: {
+            canonical,
+        },
+    };
+
+    try {
+        await fetchLicense(licenseType, licenseId, config.environment);
+    } catch (error) {
+        return errorMetadataWithCanonical;
+    }
+
+    return {
+        title: `License #${licenseId}`,
+        openGraph: {
+            title: `License #${licenseId}`,
+        },
+        alternates: {
+            canonical,
+        },
+    };
+}
 
 const fetchLicense = async (
     licenseType: 'ND' | 'MND' | 'GND',

app/server-components/LicensePage/LicensePageNodeCardWrapper.tsx

@@ -13,7 +13,7 @@ export default async function LicensePageNodeCardWrapper({
             await cachedGetNodeAvailability();
 
         if (!nodeResponse) {
-            return <ErrorAlert />;
+            return null;
         }
 
         return <NodeCard nodeResponse={nodeResponse} hasLink />;

app/server-components/Licenses/License.tsx

@@ -87,7 +87,7 @@ export default async function License({ licenseType, licenseId }: Props) {
                     <CardItem
                         label="Assign Timestamp"
                         value={
-                            !assignTimestamp ? (
+                            !assignTimestamp || assignTimestamp === '0' ? (
                                 <>
                                     <div className="block lg:hidden">
                                         <div className="text-slate-400">Not assigned</div>

app/server-components/Licenses/LicenseRewardsPoA.tsx

@@ -21,9 +21,7 @@ export default async function LicenseRewardsPoA({
             await getNodeAvailability();
 
         if (!nodeResponse) {
-            return (
-                <CardHorizontal label="Rewards" value={<SmallTag variant="banned">Error loading rewards</SmallTag>} isSmall />
-            );
+            return null;
         }
 
         const firstCheckEpoch: number = getLicenseFirstCheckEpoch(license.assignTimestamp);