✨ feat: use .env file for API url#100
Conversation
caverav
requested review from
Icksir,
iTzGooDLife,
jllanosg,
massi-ponce and
sealra
WalkthroughWalkthroughSe han realizado cambios significativos en el código del frontend para implementar el uso de variables de entorno, específicamente para la configuración de URLs de servicios. Se ha añadido un archivo Changes
Assessment against linked issues
Recent review detailsConfiguration used: CodeRabbit UI Files selected for processing (1)
Additional comments not posted (2)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
Files selected for processing (9)
- .gitignore (1 hunks)
- frontend/.env.example (1 hunks)
- frontend/Dockerfile (1 hunks)
- frontend/src/hooks/useAuth.tsx (3 hunks)
- frontend/src/routes/audits/edit/AuditRoot.tsx (4 hunks)
- frontend/src/services/audits.ts (1 hunks)
- frontend/src/services/data.ts (2 hunks)
- frontend/src/services/settings.ts (1 hunks)
- frontend/src/services/vulnerabilities.ts (1 hunks)
Additional comments not posted (9)
frontend/.env.example (1)
1-1: Archivo de ejemplo.env.exampleagregado correctamenteLa inclusión del archivo de ejemplo
.env.examplefacilita la configuración del entorno para otros desarrolladores. Buen trabajo al proporcionar una plantilla clara..gitignore (1)
12-12: Se agregó.enval.gitignoreEs una buena práctica excluir el archivo
.envdel control de versiones para proteger información sensible. Bien hecho.frontend/src/services/settings.ts (1)
1-1: Uso adecuado de variables de entorno paraAPI_URLLa actualización permite configurar la URL de la API según el entorno, lo cual mejora la flexibilidad de la aplicación. Excelente implementación.
frontend/src/hooks/useAuth.tsx (1)
4-5: Actualización de endpoints para usar variables de entornoLas modificaciones reemplazan URLs hardcodeadas con variables de entorno, mejorando la configurabilidad y el mantenimiento del código. Buen trabajo al asegurar que las llamadas a la API utilicen
VITE_API_URL.Also applies to: 39-46, 61-67
frontend/src/routes/audits/edit/AuditRoot.tsx (1)
79-81: Actualización de rutas de generación de documentos con variables de entornoHas reemplazado correctamente las URLs directas con variables de entorno, permitiendo mayor flexibilidad en diferentes entornos. Bien hecho.
Also applies to: 89-91, 99-101, 109-111
frontend/src/services/vulnerabilities.ts (1)
1-1: Excelente implementación de variables de entorno para elAPI_URL.Esto mejora la flexibilidad y facilita la configuración en diferentes entornos.
frontend/src/services/data.ts (2)
1-1: Buena práctica al utilizar variables de entorno paraAPI_URL.Esto permite una configuración más flexible entre diferentes entornos.
428-428: Buen ajuste al utilizarAPI_URLen la funcióndownloadTemplate.Esto asegura que la URL de descarga también sea configurable según el entorno.
frontend/src/services/audits.ts (1)
1-1: Excelente actualización al utilizar variables de entorno paraAPI_URL.Esto mejora la flexibilidad y facilita el despliegue en múltiples entornos.
|
xq la url debe ir con el nombre VITE_API_URL? |
Se explica bien en la documentación (https://vitejs.dev/guide/env-and-mode) pero básicamente las variables iniciadas en VITE_ se exponen al cliente, cualquier otra no |
2 participants
Descripción
Se agrega el uso de variables de entorno para las URLs de la API
Motivación y Contexto
Está mal hardcodear las URLs, fixes #99
¿Cómo ha sido probado?
Probar el flujo de la aplicación, funcionando correctamente las llamadas a la API
Tipos de cambios
Lista de verificación:
Summary by CodeRabbit
Nuevas Funciones
.env.exampleque proporciona una plantilla de configuración para las variables de entorno, facilitando la configuración del entorno de desarrollo..enven el proceso de construcción del contenedor.Mejoras
Correcciones