binarycat0 commented on code in PR #147:
URL: https://github.com/apache/polaris-tools/pull/147#discussion_r2737724999
##########
console/src/api/client.ts:
##########
@@ -20,14 +20,12 @@
import axios, { type AxiosInstance, type InternalAxiosRequestConfig } from
"axios"
import { navigate } from "@/lib/navigation"
import { REALM_HEADER_NAME } from "@/lib/constants"
+import { config } from "@/lib/config"
-// Always use relative URLs to go through the proxy (dev server or production
server)
-// This avoids CORS issues by proxying requests through the server
-// The server.ts proxy handles /api routes in production, and Vite handles
them in development
-const API_BASE_URL = ""
+const API_BASE_URL = config.POLARIS_API_URL
const MANAGEMENT_BASE_URL = `${API_BASE_URL}/api/management/v1`
const CATALOG_BASE_URL = `${API_BASE_URL}/api/catalog/v1`
-const POLARIS_BASE_URL = `${API_BASE_URL}/polaris/v1`
+const POLARIS_BASE_URL = `${API_BASE_URL}/api/catalog/polaris/v1`
Review Comment:
It might be, I didn't bring this variable in my PR, but I can rename it
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
