Merge pull request #1250 from SkynetLabs/fix-external-links-on-news-header

fixed the latest news link redirect in the news header

changelog/items/bugs-fixed/fix-external-links-on-news-header.md

@@ -0,0 +1 @@
+- fixed the latest news link redirect in the news header

packages/website/src/components/NewsHeader/NewsHeader.js

@@ -12,6 +12,7 @@ const NewsHeader = () => {
             id
             frontmatter {
               title
+              external
             }
             fields {
               slug
@@ -26,8 +27,9 @@ const NewsHeader = () => {
 
   if (!latestNews) return null; // no news
 
-  const title = latestNews.node.frontmatter.title;
+  const { frontmatter, fields } = latestNews.node;
-  const link = { to: latestNews.node.fields.slug };
+  const { title, external } = frontmatter;
+  const link = external ? { href: external } : { to: fields.slug };
 
   return (
     <div className="bg-palette-500 px-8 p-3">

packages/website/src/pages/news.js

@@ -6,39 +6,36 @@ import { NewsSummary } from "../components/News";
 import Link from "../components/Link";
 import Seo from "../components/seo";
 
-const NewsCard = ({ ...props }) => {
+const NewsCard = ({ frontmatter, fields }) => {
-  const linkProps = { to: !props.frontmatter.external && props.fields.slug, href: props.frontmatter.external };
+  const { title, external, categories, description, thumbnail, avatar, author, date } = frontmatter;
+  const link = external ? { href: external } : { to: fields.slug };
 
   return (
     <div className="flex flex-col">
-      <Link {...linkProps} className="flex items-center">
+      <Link {...link} className="flex items-center">
-        <GatsbyImage image={getImage(props.frontmatter.thumbnail)} alt={props.frontmatter.title} />
+        <GatsbyImage image={getImage(thumbnail)} alt={title} />
       </Link>
 
-      {props.frontmatter.categories && (
+      {categories && (
         <div className="mt-6">
-          {props.frontmatter.categories.map((category) => (
+          {categories.map((category) => (
             <Label key={category}>{category}</Label>
           ))}
         </div>
       )}
 
-      <Link {...linkProps} className="text-xl mt-6">
+      <Link {...link} className="text-xl mt-6">
-        {props.frontmatter.title}
+        {title}
       </Link>
 
-      {props.frontmatter.description && (
+      {description && (
-        <Link {...linkProps} className="font-content text-palette-400 mt-4">
+        <Link {...link} className="font-content text-palette-400 mt-4">
-          {props.frontmatter.description}
+          {description}
         </Link>
       )}
 
       <div className="mt-6">
-        <NewsSummary
+        <NewsSummary avatar={avatar} author={author} date={date} />
-          avatar={props.frontmatter.avatar}
-          author={props.frontmatter.author}
-          date={props.frontmatter.date}
-        />
       </div>
     </div>
   );