Azure / Azure/data-api-builder
[Enhancement]: Improved DAB CLI appinsights config usage
- Lenguaje dominante
- C#
- Estrellas
- 1.5k
- Forks
- 370
- Merge medio
- 3 d 22 h
- PR fusionados (30 d)
- 9
Descripción
### What happened?
Improved experience with adding telemetry config via CLI:
1. `add-telemetry` -> indicates you'd be adding new telemetry sections each time. Unclear that you need to use this for "update" as well. What about "delete?
2. `--app-insights-enabled` option can't be used on its own, always needs a connection string. This makes sense if all you're doing is "adding" but doesn't make sense for "update".
### Version
.11rc
### What database are you using?
Azure SQL
### What hosting model are you using?
Local (including CLI)
### Which API approach are you accessing DAB through?
REST
### Relevant log output
_No response_
### Code of Conduct
- [X] I agree to follow this project's Code of Conduct
Guía de contribución
Línea de trabajo
No se nombran archivos, pruebas ni puntos de entrada. Empieza por localizar la configuración de telemetría de DAB CLI y la gestión de las opciones de App Insights, y después aclara el comportamiento previsto para agregar, actualizar y eliminar; hecho debería significar que la CLI admite las operaciones acordadas sin requerir un connection string innecesario.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- azure, csharp
- Área
- cli, cloud, observability
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 35/100