clean up todos

Author: jderochervlk

app/routes/ApiRoute.res

@@ -110,7 +110,6 @@ let loader: ReactRouter.Loader.t<loaderData> = async args => {
   let data = {
     // TODO POST RR7: refactor this function to only return the module and not the toctree
     // or move the toc logic to this function
-    // TODO move the loader function to its own file
     try {
       await ApiDocs.getStaticProps(path)
     } catch {

src/BlogLoader.res

@@ -14,7 +14,14 @@ let transform = (mdx: Mdx.attributes): BlogApi.post => {
     frontmatter: {
       author: BlogFrontmatter.authors
       ->Array.find(author => author.username->String.includes(mdx.author))
-      ->Option.getOrThrow, // TODO: this is probably unsafe and needs to be fixed
+      ->Option.getOr({
+        // Fallback to the ReScript Team if we can't find a matching author
+        username: "rescript-team",
+        fullname: "ReScript Team",
+        role: "Core Development",
+        imgUrl: "https://pbs.twimg.com/profile_images/1358354824660541440/YMKNWE1V_400x400.png",
+        social: X("rescriptlang"),
+      }),
       co_authors: (mdx.co_authors->Nullable.getOr([]) :> array<BlogFrontmatter.author>),
       date: mdx.date,
       previewImg: mdx.previewImg,

src/components/ImageGallery.res

@@ -65,8 +65,6 @@ let make = (
       }
     })
   }
-  // TODO - Replace this with just using tailwind show and transitions?
-  // It's causing a hydration mismatch
   <div className>
     <div className="w-full" onClick>
       <HeadlessUI.Transition

src/components/Markdown.res

@@ -46,8 +46,7 @@ module UrlBox = {
   let imgEl = <img src="/hyperlink.svg" className="mr-2 inline-block" />
 
   @react.component
-  let // TODO: can this href be a Path.t somehow?
-  make = (~text: string, ~href: string, ~children: MdxLegacy.MdxChildren.t) => {
+  let make = (~text: string, ~href: string, ~children: MdxLegacy.MdxChildren.t) => {
     let content = switch classify(children) {
     | String(str) =>
       <p>
@@ -117,14 +116,12 @@ module Anchor = {
     <span className="inline group relative" title>
       <a
         title
-        // TODO RR7: the scroll margin might need some tweaking for media breakpoints
         className="scroll-mt-30 invisible text-gray-60 opacity-50 hover:opacity-100 hover:text-gray-60 hover:cursor-pointer group-hover:visible"
         href={"#" ++ id}
         id={"#" ++ id}
       >
         <Icon.Hyperlink className="inline-block align-middle text-gray-40" />
       </a>
-      // <a className="absolute -top-28" id />
     </span>
   }
 }
@@ -139,7 +136,6 @@ module H2 = {
   @react.component
   let make = (~id, ~children) => {
     // Children may not be a string
-    // TODO: RR7 - this can be improved, but I need to figure out what the other possible types are
     let title = {
       try {
         childrenToString(children)->Url.normalizeAnchor

src/components/SearchBox.res

@@ -49,7 +49,6 @@ let make = (
 
     let full = (ctrlKey ? "CTRL+" : "") ++ key
 
-    // TODO RR7 test keyboard input
     switch full {
     | "Escape" => onClear()
     | "Tab" =>

src/layouts/ApiLayout.res

@@ -76,25 +76,6 @@ let make = (
   let (isSidebarOpen, setSidebarOpen) = React.useState(_ => false)
   let toggleSidebar = () => setSidebarOpen(prev => !prev)
 
-  React.useEffect(() => {
-    // TODO: replicate this for React Router
-    // open Next.Router.Events
-    // let {Next.Router.events: events} = router
-
-    // let onChangeComplete = _url => setSidebarOpen(_ => false)
-
-    // events->on(#routeChangeComplete(onChangeComplete))
-    // events->on(#hashChangeComplete(onChangeComplete))
-
-    // Some(
-    //   () => {
-    //     events->off(#routeChangeComplete(onChangeComplete))
-    //     events->off(#hashChangeComplete(onChangeComplete))
-    //   },
-    // )
-    None
-  }, [])
-
   let navigate = ReactRouter.useNavigate()
 
   let preludeSection =

src/layouts/SidebarLayout.res

@@ -292,7 +292,6 @@ let make = (
   | None => React.null
   }
 
-  // TODO: RR7 figure this out
   let pagination = switch categories {
   | Some(categories) =>
     let items = categories->Array.flatMap(c => c.items)