fixes

Author: petercrocker

docker-compose.override.yml

@@ -11,10 +11,6 @@ services:
 
   # Mount local repository for local development
   # Used when INFRAHUB_GIT_LOCAL=true to work with local generator/transform changes
-  infrahub-git:
-    volumes:
-      - ./:/upstream
-
   task-worker:
     volumes:
       - ./:/upstream

service_catalog/Home.py

@@ -46,7 +46,8 @@ def main() -> None:
     # Initialize API client
     client = InfrahubClient(
         st.session_state.infrahub_url,
-        api_token=INFRAHUB_API_TOKEN or None
+        api_token=INFRAHUB_API_TOKEN or None,
+        ui_url=INFRAHUB_UI_URL
     )
 
     # Page title

service_catalog/pages/1_Create_DC.py

@@ -14,6 +14,7 @@
     GENERATOR_WAIT_TIME,
     INFRAHUB_ADDRESS,
     INFRAHUB_API_TOKEN,
+    INFRAHUB_UI_URL,
     InfrahubClient,
     display_error,
     display_logo,
@@ -445,7 +446,8 @@ def main() -> None:
     # Initialize API client to fetch locations
     client = InfrahubClient(
         st.session_state.infrahub_url,
-        api_token=INFRAHUB_API_TOKEN or None
+        api_token=INFRAHUB_API_TOKEN or None,
+        ui_url=INFRAHUB_UI_URL
     )
 
     # Fetch locations (cache in session state)

service_catalog/pages/2_Rack_Visualization.py

@@ -247,7 +247,9 @@ def main() -> None:
 
     # Initialize API client
     client = InfrahubClient(
-        st.session_state.infrahub_url, api_token=INFRAHUB_API_TOKEN or None
+        st.session_state.infrahub_url,
+        api_token=INFRAHUB_API_TOKEN or None,
+        ui_url=INFRAHUB_UI_URL
     )
 
     # Page title

service_catalog/pages/3_VLAN_Management.py

@@ -433,7 +433,9 @@ def main() -> None:
 
     # Initialize API client (always use "main" branch)
     client = InfrahubClient(
-        st.session_state.infrahub_url, api_token=INFRAHUB_API_TOKEN or None
+        st.session_state.infrahub_url,
+        api_token=INFRAHUB_API_TOKEN or None,
+        ui_url=INFRAHUB_UI_URL
     )
 
     # Page title

service_catalog/utils/api.py

@@ -37,15 +37,17 @@ def __init__(self, message: str, errors: List[Dict[str, Any]]):
 class InfrahubClient:
     """Client for interacting with the Infrahub API using the official SDK."""
 
-    def __init__(self, base_url: str, api_token: Optional[str] = None, timeout: int = 30):
+    def __init__(self, base_url: str, api_token: Optional[str] = None, timeout: int = 30, ui_url: Optional[str] = None):
         """Initialize the Infrahub API client.
 
         Args:
             base_url: Base URL of the Infrahub instance (e.g., "http://localhost:8000")
             api_token: Optional API token for authentication (not currently used by SDK)
             timeout: Request timeout in seconds (default: 30)
+            ui_url: Optional UI URL for generating browser links (defaults to base_url if not provided)
         """
         self.base_url = base_url.rstrip("/")
+        self.ui_url = (ui_url or base_url).rstrip("/")
         self.api_token = api_token
         self.timeout = timeout
 
@@ -588,9 +590,9 @@ def get_proposed_change_url(self, pc_id: str) -> str:
             pc_id: Proposed change ID
 
         Returns:
-            URL string for the proposed change
+            URL string for the proposed change (uses UI URL for browser access)
         """
-        return f"{self.base_url}/proposed-changes/{pc_id}"
+        return f"{self.ui_url}/proposed-changes/{pc_id}"
 
     def get_location_rows(self, branch: str = "main") -> List[Dict[str, Any]]:
         """Fetch LocationRow objects.