Post-review fixes

JohnThomson · JohnThomson · commit 6ec5148145ed · 2022-09-20T13:57:39.000-05:00
diff --git a/public/index.html b/public/index.html
@@ -48,10 +48,11 @@
     </head>
     <body>
         <noscript>You need to enable JavaScript to run this app.</noscript>
-        <div id="root">loading...</div>
+        <div id="root">Loading...</div>
         <!--
       This HTML file is a template.
-      If you open it directly in the browser, you will see an empty page.
+      If you open it directly in the browser, you will just see the Loading message.
+      (It's there so something can be seen while fetching the code.)
 
       You can add webfonts, meta tags, or analytics to this file.
       The build step will place the bundled scripts into the <body> tag.
diff --git a/src/connection/LibraryQueryHooks.ts b/src/connection/LibraryQueryHooks.ts
@@ -112,8 +112,7 @@ export function useGetCleanedAndOrderedLanguageList(): ILanguage[] {
 export function useGetTagList(): string[] {
     // I'd prefer to use useAppHosted but it crashes, I think because this code runs outside the
     // scope where it works.
-    const appHosted = window.location.pathname.startsWith("/app-hosted-v1");
-    console.log("useGetTagList: " + appHosted);
+    const appHosted = isAppHosted();
     const axiosResult = useLibraryQueryWithCount(
         "tag",
         {
diff --git a/src/connection/ParseServerConnection.ts b/src/connection/ParseServerConnection.ts
@@ -42,6 +42,9 @@ const local: IConnection = {
 
 export function getConnection(): IConnection {
     const result = prod;
+    // Some previous code which we don't need now...only one case returns anything but prod...
+    // but I think it's worth keeping to document the possibilities that we explicitly
+    // decided should be prod.
     // if (
     //     window.location.hostname === "bloomlibrary.org" ||
     //     window.location.hostname === "next.bloomlibrary.org" ||
