docs: integrate contentful to F36 website #1973
Conversation
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/contentful-apps/forma-36/2VBYHyJKTgQR2mKtLU8ip62owWDZ |
|
burakukula
requested review from
a team,
mshaaban0,
denkristoffer and
Lelith
as code owners
burakukula
changed the title
| `query { | ||
| kbAppArticleCollection(where: { slug_exists: true }) { | ||
| items { | ||
| ${ARTICLE_GRAPHQL_FIELDS} |
There was a problem hiding this comment.
I'm gonna use simpler query over here. only thing we need here is slug, but on the separate PR.
| for (const entry of links.entries.block) { | ||
| entryMap.set(entry.sys.id, entry); | ||
| } | ||
| return { |
There was a problem hiding this comment.
Do we not use eslint anymore? For me it's all red
packages/website/pages/[...slug].tsx
Outdated
| @@ -58,76 +53,142 @@ export default function ComponentPage({ | |||
| </PropsContextProvider> | |||
| </> | |||
| ); | |||
| }; | |||
|
|
|||
| // TODO: mrege Heading and getToC to getTableOfContents from 'mdx-utils' | |||
There was a problem hiding this comment.
Do you plan to do it in a separate story?
packages/website/pages/[...slug].tsx
Outdated
| // [WIP]: I started adding stronger types to this function but I could not finish | ||
| // that's why I renamed some stuff here, I was trying to follow this: | ||
| // https://nextjs.org/docs/basic-features/typescript#static-generation-and-server-side-rendering | ||
| export const getStaticProps = async (context: { |
There was a problem hiding this comment.
I would separate some parts into functions because right now it's a bit hard to read and understand. For example the part with shortIntroText can be isolated
* refactor: improve ComponentPage typing * refactor: move ToC utils to same file
Co-authored-by: Gui Santos <guilherme.goncalvessantos@gmail.com>
Adding contentful to Forma 36 website:
TODO in the future: