fix: make Algolia and FeedbackFarm variables optional (#2016)

* I think this is okay as long as Algolia is purely client-side loaded
  as it is now. I don't think this will work if we need to render
  Algolia server-side
* Also remove them from the GitHub action build for simplicity
* Note this was already the case for Segment

.github/workflows/ci-default.yml

@@ -13,10 +13,6 @@ env:
   NEXT_PUBLIC_DB_GRAPHQL_URL: ${{ vars.NEXT_PUBLIC_DB_GRAPHQL_URL }}
   HASURA_URL: ${{ vars.HASURA_URL }}
   OSO_API_KEY: ${{ secrets.OSO_API_KEY }}
-  NEXT_PUBLIC_ALGOLIA_APPLICATION_ID: "test"
-  NEXT_PUBLIC_ALGOLIA_API_KEY: "test"
-  NEXT_PUBLIC_ALGOLIA_INDEX: "test"
-  NEXT_PUBLIC_FEEDBACK_FARM_ID: "test"
   # Docs variables
   DOCS_URL: "https://docs.opensource.observer"
   DOCS_ALGOLIA_APP_ID: "test"

apps/frontend/.env.local.example

@@ -29,7 +29,7 @@ CLICKHOUSE_URL=
 CLICKHOUSE_USERNAME=
 CLICKHOUSE_PASSWORD=
 CLICKHOUSE_DB_NAME=
-### Algolia for search (required)
+### Algolia for search (optional)
 NEXT_PUBLIC_ALGOLIA_APPLICATION_ID=
 NEXT_PUBLIC_ALGOLIA_API_KEY=
 NEXT_PUBLIC_ALGOLIA_INDEX=

apps/frontend/lib/config.ts

@@ -45,19 +45,10 @@ export const CLICKHOUSE_USERNAME = process.env.CLICKHOUSE_USERNAME ?? "MISSING";
 export const CLICKHOUSE_PASSWORD = process.env.CLICKHOUSE_PASSWORD ?? "MISSING";
 export const CLICKHOUSE_DB_NAME = process.env.CLICKHOUSE_DB_NAME ?? "default";
 
-export const ALGOLIA_APPLICATION_ID = requireEnv(
-  process.env.NEXT_PUBLIC_ALGOLIA_APPLICATION_ID,
-  "NEXT_PUBLIC_ALGOLIA_APPLICATION_ID",
-);
-export const ALGOLIA_API_KEY = requireEnv(
-  process.env.NEXT_PUBLIC_ALGOLIA_API_KEY,
-  "NEXT_PUBLIC_ALGOLIA_API_KEY",
-);
-export const ALGOLIA_INDEX = requireEnv(
-  process.env.NEXT_PUBLIC_ALGOLIA_INDEX,
-  "NEXT_PUBLIC_ALGOLIA_INDEX",
-);
-export const FEEDBACK_FARM_ID = requireEnv(
-  process.env.NEXT_PUBLIC_FEEDBACK_FARM_ID,
-  "NEXT_PUBLIC_FEEDBACK_FARM_ID",
-);
+export const ALGOLIA_APPLICATION_ID =
+  process.env.NEXT_PUBLIC_ALGOLIA_APPLICATION_ID ?? "MISSING";
+export const ALGOLIA_API_KEY =
+  process.env.NEXT_PUBLIC_ALGOLIA_API_KEY ?? "MISSING";
+export const ALGOLIA_INDEX = process.env.NEXT_PUBLIC_ALGOLIA_INDEX ?? "MISSING";
+export const FEEDBACK_FARM_ID =
+  process.env.NEXT_PUBLIC_FEEDBACK_FARM_ID ?? "MISSING";