chore(greenhouse): add org to subdomain in non-dev environments (#1259)

* chore(greenhouse): add org to subdomain in non-dev environments

Co-authored-by: Arturo <a.reuschenbach.puncernau@sap.com>

* refactor(greenhouse): use generic way to detect environment

---------

Co-authored-by: Arturo <a.reuschenbach.puncernau@sap.com>

Author: taymoor89

.changeset/ready-trees-chew.md

@@ -0,0 +1,5 @@
+---
+"@cloudoperators/juno-app-greenhouse": patch
+---
+
+For non-development environment always put organization name as subdomain.

.github/scripts/build-vite-app.sh

@@ -8,7 +8,7 @@ package_name=$(jq -r '.name' $PACKAGE_PATH/package.json)
 app_props_url=$PREVIEW_URL/$TARGET_FOLDER/appProps.json
 
 # Run build using turbo
-VITE_APP_PROPS_URL=$app_props_url pnpx turbo run build:static --filter $package_name -- --base ./
+NODE_ENV=$NODE_ENV VITE_APP_PROPS_URL=$app_props_url pnpx turbo run build:static --filter $package_name -- --base ./
 
 # Copy build folder to deploy path
 mkdir -p "$DEPLOY_PATH/$TARGET_FOLDER"

.github/workflows/deploy-pr-preview.yaml

@@ -151,6 +151,7 @@ jobs:
           TARGET_FOLDER: greenhouse
           APP_PROPS_BASE64: ${{ secrets.GREENHOUSE_APP_PROPS_BASE64 }}
           PREVIEW_URL: ${{ env.PREVIEW_URL }}
+          NODE_ENV: development
         continue-on-error: true
 
       - name: Build EXAMPLE if changes detected

apps/greenhouse/src/components/AuthProvider.tsx

@@ -7,26 +7,33 @@ import React, { createContext, useContext, useState, useMemo, useRef, useEffect
 import { oidcSession, mockedSession, tokenSession } from "@cloudoperators/juno-oauth"
 
 const setOrganizationToUrl = (groups: any, enableHashedRouting: boolean) => {
-  const orgString = groups?.find((g: any) => g.indexOf("organization:") === 0)
-  if (orgString) {
-    const name = orgString.split(":")[1]
-    let url = new URL(window.location.href)
-
-    // Check if the organization name is already in the URL path or hash then don't change it
-    const currentPath = enableHashedRouting ? url.hash.replace("#/", "") : url.pathname
-    const currentFirstSegment = currentPath.split("/").filter(Boolean)[0]
-    if (name === currentFirstSegment) return
-
-    // if enableHashedRouting is true, set the hash, otherwise set the pathname
-    const pathWithOrg = `/${name}`
-    if (enableHashedRouting) {
-      url.hash = pathWithOrg
-    } else {
-      url.pathname = pathWithOrg
-    }
-    // @ts-expect-error TS(2345): Argument of type 'null' is not assignable to param... Remove this comment to see the full error message
-    window.history.replaceState(null, null, url.href)
+  const orgName = groups?.find((g: any) => g.startsWith("organization:"))?.split(":")[1]
+
+  if (!orgName) return
+
+  let url = new URL(window.location.href)
+
+  // In non-dev environments, set the organization as subdomain
+  if (!import.meta.env.DEV) {
+    url.hostname = `${orgName}.${url.hostname}`
+    window.location.href = url.href
+    return
+  }
+
+  // Check if the organization name is already in the URL path or hash then don't change it
+  const currentPath = enableHashedRouting ? url.hash.replace("#/", "") : url.pathname
+  const currentFirstSegment = currentPath.split("/").filter(Boolean)[0]
+  if (orgName === currentFirstSegment) return
+
+  // if enableHashedRouting is true, set the hash, otherwise set the pathname
+  const pathWithOrg = `/${orgName}`
+  if (enableHashedRouting) {
+    url.hash = pathWithOrg
+  } else {
+    url.pathname = pathWithOrg
   }
+  // @ts-expect-error TS(2345): Argument of type 'null' is not assignable to param... Remove this comment to see the full error message
+  window.history.replaceState(null, null, url.href)
 }
 
 function resolveMockAuth(value: any) {