Separar download sat #26
Conversation
Se agrega soporte para obtener el detalle de conceptos en los reportes, por ejemplo:
{emisor_rfc}|{serie}{folio}|{noIdentificacion}|{descripcion}|{unidad}|{cantidad}|{valorUnitario}|{importe}
Si agrega cualquier campo del detalle, se omite la validación de los sellos y del SAT
- Habilitar la extensión autodoc - Crear la sección Referencia
- Qué código está involucrado - Los pasos generales
- Por ahora solo los que tienen que ver con la descarga de CFDIs
- Es copia sin refactorizar de _download_sat() - Queda en pyutil.py como las demás herramientas
- Crear descarga.py como script para ejecutar desde consola - Crear pwd.sample como plantilla del archivo de credenciales
|
Ya se agregaron cambios en esta rama, que crean DescargaSAT y descarga.py. Si quieren probar descarga.py, es un script de consola que usa DescargaSAT. |
- Alineando con la definición propuesta en el issue LinuxCabal#25
- Ayuda con -h - Se ofrecen valores predeterminados
- La carpeta destino tiene que ser una ruta absoluta. Si es relativa, el navegador la ignora y utiliza su valor predeterminado de descargas :(
- Las validaciones que hace - El diccionario que construye - La tupla que regresa
- Los valores predeterminados igual que en la aplicación admincfdi
- El uso de uuid cambia type_search a 1
- Se preserva self para referir a _download_sat_month() que se va a agregar ahora - Ya podemos refactorizar _download_sat() porque podemos probar funcionalmente todas las opciones de búsqueda
- Es copia sin refactorizar de _download_sat_month() - Como método de DescargaSAT
|
Se agregaron más cambios: descarga.py ya tiene opciones de línea de comando para todos los tipos de búsquedas, pueden listar las opciones así: Si quieren probar descarga.py, hagan pull de esta rama en mi fork. Lo he estado usando para probar funcionalmente los cambios y está muy padre hacer las descagas desde bash ;) Espero que ya pronto lleguemos al cambio para que admincfdi.py use directamente DescargaSAT |
- Se hará innecesaria mediante refactorización en DescargaSAT
|
Buen día Quiero probar la descarga.py... ¿como hacer el pull de la rama? |
|
@mr-E: Debería decir hacer checkout, en lugar pull. Si estás dentro del repositorio admin-cfdi en tu PC, agrega una referencia remota a mi fork del proyecto así: Actualiza las referencias en tu repositorio: Ahora sí, haz checkout de la rama separar-dowload-sat: Estando en esta rama, deberás ver descarga.py en tu repositorio. Algo que no he resuelto aún, es porqué FireFox ignora el valor de --carpeta-destino si es una ruta relativa... usa una ruta absoluta como ~/cfdi, y crea esa carpeta antes de correr descarga.py. |
|
Cuando hago Creo recordar que hay que pasarle rutas abosultas a Firefox... |
|
Eran unos cambios que había hecho... probando descarga.py |
|
Prueba exitosa, quedo muy padre... |
- Usar get_dates, get_days, sleep - Remover del mock de aplication
- Se usan el atributo SAT, page_init y page_cfdi - Remover del mock de aplication
- El atributo status es llamado para enviar mensajes - Por omisión los mensajes se envían a stdout - Remover del mock de aplication
- El atributo progress es llamado para proporcionar valor y máximo a un control gráfico de tipo barra de progreso - Por omisión se envían valor y máximo a stdout - Remover pb del mock de aplication
- Remover el mock
|
Intente probar "descarga.py" pero me falta: Usando pip intente instalar "unittest", "unittest2" y "mock", pero no funciona. No lo encuentra y menciona en el log que va a ignorar unittest2 : |
|
unittest es parte de la librería estandar, pero mock fue introducido en Python 3.3, eso es lo que posiblemente te está faltando. Si tienes Python 3.2 por ejemplo, podrías intentar y modifca esos dos import para que digan: Pero hay una opción mejor: esta mañana subí cambios que removieron el uso de unittest.mock en descarga.py :) Simplemente actualiza tu repositorio así: |
|
+1 Gracias! |
- Agregar los métodos msg_user() y progress() como wrapper a los widgets - Usar estos métodos en los parámetros opcionales status_callback y download_callback
- _download_sat() y _download_sat_month() ya están incluidas en pyutil.DescargaSAT
ghost
merged commit
|
La mezcla a master se deshizo, no era lo que se deseaba y no nos dimos cuenta, y se rehizo la mezcla a la rama develop. |
Esta rama derivada de develop es para realizar el issue #25. Empieza con agregados a la documentación que son necesarios, se agregarán más cambios en código esta tarde al final de la sesión de TDD en LinuxCabal.
Una vez que esté completa, esta rama podría mezclarse en un release-0.2.7 o subsequente.