delete pages folder

Author: jderochervlk

app/routes.res

@@ -21,8 +21,6 @@ let default = [
   route("syntax-lookup", "./routes/SyntaxLookupRoute.mjs", ~options={id: "syntax-lookup"}),
   route("blog", "./routes/BlogRoute.mjs", ~options={id: "blog-index"}),
   route("blog/archived", "./routes/BlogRoute.mjs", ~options={id: "blog-archived"}),
-  // TODO RR7 get the api index to work with the same template
-  // route("docs/manual/api", "./routes/ApiRoute.mjs", ~options={id: "api-index"}),
   route("docs/manual/api/stdlib", "./routes/ApiRoute.mjs", ~options={id: "api-stdlib"}),
   route("docs/manual/api/introduction", "./routes/ApiRoute.mjs", ~options={id: "api-intro"}),
   route("docs/manual/api/belt", "./routes/ApiRoute.mjs", ~options={id: "api-belt"}),

app/routes/ApiRoute.res

@@ -78,7 +78,7 @@ let groupItems = apiDocs => {
       let parentNode =
         rootItems->Array.find(item => item.children->Array.some(node => node.name === parentName))
 
-      // TODO: this can probably be refactored
+      // TODO POST RR7: this can probably be refactored
       parentNode->Option.forEach(
         parentNode =>
           parentNode.children->Array.forEach(
@@ -108,7 +108,7 @@ let loader: ReactRouter.Loader.t<loaderData> = async args => {
   let toctree = groupItems(apiDocs)
 
   let data = {
-    // TODO RR7: refactor this function to only return the module and not the toctree
+    // 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 {

app/routes/MdxRoute.res

@@ -134,7 +134,6 @@ let loader: Loader.t<loaderData> = async ({request}) => {
 
   let mdx = await loadMdx(request, ~options={remarkPlugins: [Mdx.gfm]})
 
-  // TODO: actually render the blog pages
   if pathname->String.includes("blog") {
     let posts = blogPosts()
 
@@ -179,8 +178,8 @@ let loader: Loader.t<loaderData> = async ({request}) => {
       }
     }
 
-    // TODO: this can be optionally called if we need markdown
-    // TODO: extract this out into a separate function
+    // TODO POST RR7: extract this out into a separate function
+    // it can probably be cached or something
     let fileContents = await (await allMdx())
     ->Array.filter(mdx => (mdx.path :> string)->String.includes(pathname))
     ->Array.get(0)
@@ -244,7 +243,7 @@ let default = () => {
 
   let {entries, categories} = loaderData
 
-  // TODO: get actual meta categories working
+  // TODO RR7: get actual meta categories working
   let metaTitleCategory =
     (pathname :> string)->String.includes("docs/manual")
       ? "ReScript Language Manual"

app/routes/SyntaxLookupRoute.res

@@ -9,6 +9,7 @@ let convert = (mdx: Mdx.attributes): SyntaxLookup.item => {
     id: mdx.id->Option.getOrThrow,
     keywords: mdx.keywords->Option.getOr([]),
     name: mdx.name->Option.getOrThrow,
+    // TODO RR7 this can be removed I think
     children: <div> {React.string("TODO: render MDX here")} </div>,
     status: SyntaxLookup.Status.fromString(mdx.status->Option.getOr("active")),
     summary: mdx.summary->Option.getOrThrow,