Release 0.3 #91
Merged
Release 0.3 #91
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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
- 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 hará innecesaria mediante refactorización en DescargaSAT
- 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
Fix - Corrección al obtener los decimales
Fix - admin-cfdi.pyw
- Mostrar la ventana principal de admin-cfdi. A 50% de escala pero permite agrandar a 100% haciendo clic. - Mencionar ambas aplicaciones de línea de comando, mostrando un ejemplo de su salida
Agregar contenido a la Introducción
…te la documentación
Quitar acento en html_title en conf.py para que …
Se regresa a 2011 como año de inicio para descargas
…ingles o en español
Fix - Ahora se identifica correctamente …
- Se verifica el valor regresado por DescargaSAT.connect() y se cubre el caso de falla.
Pruebas connect
- Agregar parámetros opcionales hora_final y minuto_final a DescargaSAT.search() - Se descartan ceros a la izquierda, de lo contrario la página del SAT ignora el valor. Se explica en la documentación - Agregar una prueba funcional - Agregar ambos parámetros opcionales a descarga-cfdi - admin-cfdi y descarga-cfdi pasan ambos parámetros a DescargaSAT.search()
|
El 13 de febrero de 2016, 8:27, Patricio Paez [email protected]
http://pixelab.com.mx Carlos Omar Briseño Gutierrez |
|
Creo que si la página tiene soporte para hora, minuto y segundo, lo más sencillo es solo trasladar dicho soporte y que el emisor decida en que rangos los usa... ¿no creen? Excelente... ¿mezclamos?... |
|
Ok, incluiremos las opciones para especificar segundo_inicial y segundo_final, con valores por omisión 0 y 59. Todavía no mezclar... aún faltan cambios, aviso cuando estén listos. |
- Agregar parámetros opcionales hora_inicial y minuto_inicial a DescargaSAT.search() - Agregar una prueba funcional - Agregar ambos parámetros opcionales a descarga-cfdi - admin-cfdi y descarga-cfdi pasan ambos parámetros a DescargaSAT.search()
- Agregar parámetros opcionales segundo_inicial y segundo_final a DescargaSAT.search() - Agregar una prueba funcional - Agregar ambos parámetros opcionales a descarga-cfdi - admin-cfdi y descarga-cfdi pasan ambos parámetros a DescargaSAT.search()
- El método anterior convierte '0' en '' - Cubrir con prueba unitaria
|
Listos los cambios adicionales, las pruebas funcionales pasan en Debian y W10. |
|
Excelente... |
ghost
merged commit
This pull request was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hola,
Como que ya era hora... este es un intento de llegar al tan esperado milestone 'Release 0.3' :) Se incorporan los cambios recientes en la rama master, y se agrega lo necesario para soportar hora además de fecha en las búsquedas; faltan unos todavía algunos cambios que están en proceso, para la hora inicial.
Una duda: ¿Para quienes reciben grandes volúmenes de CFDIs, espeficiar horas y minutos es suficiente? Es decir, ¿Una ventana de 1 minuto es suficientemente pequeña? ¿Qué pasa si se dejan como constantes segundos inciales a 0 y segundos finales a 59 y se permite especificar horas y minutos solamente?