Skip to content

Conversation

@github-actions
Copy link

Improvements

Technical

@github-actions
Copy link
Author

github-actions bot commented Dec 23, 2025

Changelog

- [#380] Track chatbot semantic search costs and performance [#414] @sahilds1
- Update VITE_API_BASE_URL to point to the new prod url [#429] @TineoC
- sanitizer @AkhilRB0204
- refactor: use relative URLs and centralize API endpoints (closes #431) [#432] @TineoC
- ci: refactor pipelines for continuous deployment to sandbox [#442] @TineoC

TineoC and others added 10 commits December 28, 2025 11:54
Update VITE_API_BASE_URL to point to the new prod url
- Introduced a new `endpoints.ts` file to centralize all API endpoint paths for better maintainability and type safety.
- Updated various components and services to utilize the new centralized endpoints, enhancing consistency across the codebase.
- Created a comprehensive `API_ENDPOINTS_REFACTORING.md` guide detailing the refactoring process and usage patterns for the new endpoints.
- Removed hardcoded URLs in favor of the centralized configuration, improving code readability and reducing the risk of errors.
- Update Django URLs to serve all APIs under /api/ prefix
- Change frontend to use relative URLs (empty baseURL) instead of environment-specific domains
- Create centralized endpoints.ts for maintainable API URL management
- Update all frontend components to use centralized endpoints
- Remove all VITE_API_BASE_URL and REACT_APP_API_BASE_URL dependencies
- Add helper functions for dynamic endpoints with parameters

This ensures the same Docker image works in both production and sandbox
environments without requiring environment-specific configuration.

Fixes:
- Frontend calling old domain (balancer.live.k8s.phl.io)
- API calls failing after domain migration
- /login and /adminportal pages not working

Closes #431
refactor: use relative URLs and centralize API endpoints (closes #431)
ci: refactor pipelines for continuous deployment to sandbox
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants