aboutsummaryrefslogtreecommitdiffstats
path: root/src/pages/sujet
diff options
context:
space:
mode:
authorArmand Philippot <git@armandphilippot.com>2023-12-14 15:30:34 +0100
committerArmand Philippot <git@armandphilippot.com>2023-12-14 16:30:04 +0100
commit7063b199b4748a9c354ed37e64cdc84c512f2c0c (patch)
tree7506c3003c56b49a248e9adb40be610780bb540e /src/pages/sujet
parent85c4c42bd601270d7be0f34a0767a34bb85e29bb (diff)
refactor(pages): rewrite helpers to output schema in json-ld format
* make sure url are absolutes * nest breadcrumb schema in webpage schema * trim HTML tags from content/description * use a regular script instead of next/script (with the latter the schema is not updated on route change) * place the script in document head * add keywords, wordCount and readingTime keys in BlogPosting schema * fix breadcrumbs in search page (without query) * add tests (a `MatchInlineSnapshot` will be better but Prettier 3 is not supported yet)
Diffstat (limited to 'src/pages/sujet')
-rw-r--r--src/pages/sujet/[slug].tsx50
1 files changed, 18 insertions, 32 deletions
diff --git a/src/pages/sujet/[slug].tsx b/src/pages/sujet/[slug].tsx
index 9d42644..af78185 100644
--- a/src/pages/sujet/[slug].tsx
+++ b/src/pages/sujet/[slug].tsx
@@ -4,7 +4,6 @@ import type { GetStaticPaths, GetStaticProps } from 'next';
import Head from 'next/head';
import NextImage from 'next/image';
import { useRouter } from 'next/router';
-import Script from 'next/script';
import { useIntl } from 'react-intl';
import {
getLayout,
@@ -37,10 +36,10 @@ import { CONFIG } from '../../utils/config';
import {
getLinksItemData,
getPostsWithUrl,
- getSchemaJson,
- getSinglePageSchema,
- getWebPageSchema,
+ getSchemaFrom,
+ getWebPageGraph,
slugify,
+ trimHTMLTags,
} from '../../utils/helpers';
import { loadTranslation, type Messages } from '../../utils/helpers/server';
import {
@@ -87,27 +86,16 @@ const TopicPage: NextPageWithLayout<TopicPageProps> = ({ data }) => {
website: officialWebsite,
} = meta;
- const webpageSchema = getWebPageSchema({
- description: seo.description,
- locale: CONFIG.locales.defaultLocale,
- slug,
- title: seo.title,
- updateDate: dates.update,
- });
- const articleSchema = getSinglePageSchema({
- cover: cover?.src,
- dates,
- description: intro,
- id: 'topic',
- kind: 'page',
- locale: CONFIG.locales.defaultLocale,
- slug,
- title,
- });
- const schemaJsonLd = getSchemaJson([
- webpageSchema,
- articleSchema,
- breadcrumbSchema,
+ const jsonLd = getSchemaFrom([
+ getWebPageGraph({
+ breadcrumb: breadcrumbSchema,
+ copyrightYear: new Date(dates.publication).getFullYear(),
+ cover: cover?.src,
+ dates,
+ description: trimHTMLTags(intro),
+ slug,
+ title,
+ }),
]);
const messages = {
@@ -157,14 +145,12 @@ const TopicPage: NextPageWithLayout<TopicPageProps> = ({ data }) => {
<meta property="og:type" content="article" />
<meta property="og:title" content={title} />
<meta property="og:description" content={intro} />
+ <script
+ // eslint-disable-next-line react/no-danger
+ dangerouslySetInnerHTML={{ __html: JSON.stringify(jsonLd) }}
+ type="application/ld+json"
+ />
</Head>
- <Script
- // eslint-disable-next-line react/jsx-no-literals -- Id allowed
- id="schema-project"
- type="application/ld+json"
- // eslint-disable-next-line react/no-danger -- Necessary for schema
- dangerouslySetInnerHTML={{ __html: JSON.stringify(schemaJsonLd) }}
- />
<PageHeader
heading={
<>