fix: ssr always true

fern-api · Dec 20, 2024 · fc0cebb · fc0cebb
1 parent ea1b424
commit fc0cebb
Show file tree

Hide file tree

Showing 7 changed files with 32 additions and 61 deletions.
diff --git a/packages/fern-docs/ui/src/api-reference/ApiEndpointPage.tsx b/packages/fern-docs/ui/src/api-reference/ApiEndpointPage.tsx
@@ -1,7 +1,7 @@
 import type * as FernNavigation from "@fern-api/fdr-sdk/navigation";
 import { EMPTY_OBJECT } from "@fern-api/ui-core-utils";
 import { useSetAtom } from "jotai";
-import { useEffect } from "react";
+import { ReactNode, useEffect } from "react";
 import { useNavigationNodes, useWriteApiDefinitionAtom } from "../atoms";
 import { ALL_ENVIRONMENTS_ATOM } from "../atoms/environment";
 import { BottomNavigationNeighbors } from "../components/BottomNavigationNeighbors";
@@ -14,15 +14,11 @@ import {
   isApiPackageContentNode,
 } from "./ApiPackageContent";
 
-export declare namespace ApiEndpointPage {
-  export interface Props {
-    content: DocsContent.ApiEndpointPage;
-  }
-}
-
-export const ApiEndpointPage: React.FC<ApiEndpointPage.Props> = ({
+export default function ApiEndpointPage({
   content,
-}) => {
+}: {
+  content: DocsContent.ApiEndpointPage;
+}): ReactNode {
   useWriteApiDefinitionAtom(content.apiDefinition);
 
   // TODO: Why are we doing this here?
@@ -72,4 +68,4 @@ export const ApiEndpointPage: React.FC<ApiEndpointPage.Props> = ({
       </div>
     </ApiPageContext.Provider>
   );
-};
+}
diff --git a/packages/fern-docs/ui/src/api-reference/ApiReferencePage.tsx b/packages/fern-docs/ui/src/api-reference/ApiReferencePage.tsx
@@ -1,3 +1,4 @@
+import { ReactNode } from "react";
 import {
   useIsReady,
   useNavigationNodes,
@@ -8,15 +9,11 @@ import { DocsContent } from "../resolver/DocsContent";
 import { BuiltWithFern } from "../sidebar/BuiltWithFern";
 import { ApiReferenceContent } from "./ApiReferenceContent";
 
-export declare namespace ApiReferencePage {
-  export interface Props {
-    content: DocsContent.ApiReferencePage;
-  }
-}
-
-export const ApiReferencePage: React.FC<ApiReferencePage.Props> = ({
+export default function ApiReferencePage({
   content,
-}) => {
+}: {
+  content: DocsContent.ApiReferencePage;
+}): ReactNode {
   const hydrated = useIsReady();
 
   useWriteApiDefinitionAtom(content.apiDefinition);
@@ -49,4 +46,4 @@ export const ApiReferencePage: React.FC<ApiReferencePage.Props> = ({
       </div>
     </ApiPageContext.Provider>
   );
-};
+}
diff --git a/packages/fern-docs/ui/src/changelog/ChangelogEntryPage.tsx b/packages/fern-docs/ui/src/changelog/ChangelogEntryPage.tsx
@@ -6,7 +6,7 @@ import { useHref } from "../hooks/useHref";
 import { Markdown } from "../mdx/Markdown";
 import { DocsContent } from "../resolver/DocsContent";
 
-export function ChangelogEntryPage({
+export default function ChangelogEntryPage({
   content,
 }: {
   content: DocsContent.ChangelogEntryPage;

diff --git a/packages/fern-docs/ui/src/changelog/ChangelogPage.tsx b/packages/fern-docs/ui/src/changelog/ChangelogPage.tsx
@@ -39,7 +39,7 @@ function getOverviewMdx(
     : undefined;
 }
 
-export function ChangelogPage({
+export default function ChangelogPage({
   content,
 }: {
   content: DocsContent.ChangelogPage;

diff --git a/packages/fern-docs/ui/src/docs/DocsMainContent.tsx b/packages/fern-docs/ui/src/docs/DocsMainContent.tsx
@@ -6,54 +6,30 @@ import { useFeatureFlags, useIsReady } from "../atoms";
 import { FernErrorBoundary } from "../components/FernErrorBoundary";
 import type { DocsContent } from "../resolver/DocsContent";
 
-const MarkdownPage = dynamic(
-  () => import("./MarkdownPage").then(({ MarkdownPage }) => MarkdownPage),
-  {
-    ssr: true,
-  }
-);
+const MarkdownPage = dynamic(() => import("./MarkdownPage"), { ssr: true });
 
 const ApiReferencePage = dynamic(
-  () =>
-    import("../api-reference/ApiReferencePage").then(
-      ({ ApiReferencePage }) => ApiReferencePage
-    ),
+  () => import("../api-reference/ApiReferencePage"),
   { ssr: true }
 );
 
 const ApiEndpointPage = dynamic(
-  () =>
-    import("../api-reference/ApiEndpointPage").then(
-      ({ ApiEndpointPage }) => ApiEndpointPage
-    ),
+  () => import("../api-reference/ApiEndpointPage"),
   { ssr: true }
 );
 
-const ChangelogPage = dynamic(
-  () =>
-    import("../changelog/ChangelogPage").then(
-      ({ ChangelogPage }) => ChangelogPage
-    ),
-  {
-    ssr: true,
-  }
-);
+const ChangelogPage = dynamic(() => import("../changelog/ChangelogPage"), {
+  ssr: true,
+});
 
 const ChangelogEntryPage = dynamic(
-  () =>
-    import("../changelog/ChangelogEntryPage").then(
-      ({ ChangelogEntryPage }) => ChangelogEntryPage
-    ),
+  () => import("../changelog/ChangelogEntryPage"),
   { ssr: true }
 );
 
-const FeedbackPopover = dynamic(
-  () =>
-    import("../feedback/FeedbackPopover").then(
-      ({ FeedbackPopover }) => FeedbackPopover
-    ),
-  { ssr: false }
-);
+const FeedbackPopover = dynamic(() => import("../feedback/FeedbackPopover"), {
+  ssr: true,
+});
 
 const DocsMainContentRenderer = memo(
   ({ content }: { content: DocsContent }) => {

diff --git a/packages/fern-docs/ui/src/docs/MarkdownPage.tsx b/packages/fern-docs/ui/src/docs/MarkdownPage.tsx
@@ -1,13 +1,13 @@
-import { ReactElement } from "react";
+import { ReactNode } from "react";
 import { useWriteApiDefinitionsAtom } from "../atoms";
 import { LayoutEvaluator } from "../layouts/LayoutEvaluator";
 import { DocsContent } from "../resolver/DocsContent";
 
-interface MarkdownPageProps {
+export default function MarkdownPage({
+  content,
+}: {
   content: DocsContent.MarkdownPage;
-}
-
-export function MarkdownPage({ content }: MarkdownPageProps): ReactElement {
+}): ReactNode {
   useWriteApiDefinitionsAtom(content.apis);
   return <LayoutEvaluator {...content} />;
 }
diff --git a/packages/fern-docs/ui/src/feedback/FeedbackPopover.tsx b/packages/fern-docs/ui/src/feedback/FeedbackPopover.tsx
@@ -74,7 +74,7 @@ type SelectionTextToolbarProps = {
 //     );
 // };
 
-export const FeedbackPopover = forwardRef<
+const FeedbackPopover = forwardRef<
   SelectionTextToolbarElement,
   SelectionTextToolbarProps
 >(({ children }, forwardedRef) => {
@@ -249,3 +249,5 @@ export const FeedbackPopover = forwardRef<
 });
 
 FeedbackPopover.displayName = "FeedbackPopover";
+
+export default FeedbackPopover;