Saas and mobile fixes#1490
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds '127.0.0.1' to the saasHostnames array in the development environment configuration to fix issues related to SaaS and mobile functionality.
- Adds '127.0.0.1' to the saasHostnames array in environment.dev.ts
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| apiRoot: "https://app.rocketadmin.com/api", | ||
| saasURL: "https://app.rocketadmin.com", | ||
| saasHostnames: ['localhost'], | ||
| saasHostnames: ['localhost', '127.0.0.1'], |
There was a problem hiding this comment.
Adding '127.0.0.1' to saasHostnames in the dev environment is a good fix, but the same addition should also be made to environment.saas.ts and environment.saas-prod.ts for consistency. Both of those files currently have saasHostnames arrays with 'localhost' but are missing '127.0.0.1'. This inconsistency could lead to different behavior when the application runs on 127.0.0.1 vs localhost in different environments, even though they resolve to the same address.
No description provided.