[IMP] l10n_es_partner: Añadido script para actualizar info bancos

[ismaelcj]

Se ha creado un script que actualiza el archivo data_banks.xml a partir de un archivo descargado de la web del Banco de España.

[pedrobaeza]

Ismael, puedes quitar el "All Rights Reserved". Es una denominación anticuada, sin efecto "mercantil", y que además, sobrepasa los 80 caracteres del PEP8.

[pedrobaeza]

Buenas, Ismael,

Gracias por corregir las cosas. La próxima vez, pon por favor un comentario aquí para saber que lo has hecho, ya que si no, no nos llega aviso.

Por otro lado, aún siguen los siguientes errores PEP8 en Travis:

./l10n_es_partner/gen_src/gen_data_banks.py:6:68: W291 trailing whitespace
./l10n_es_partner/gen_src/gen_data_banks.py:10:80: E501 line too long (80 > 79 characters)
./l10n_es_partner/gen_src/gen_data_banks.py:38:80: E501 line too long (84 > 79 characters)
./l10n_es_partner/gen_src/gen_data_banks.py:47:80: E501 line too long (88 > 79 characters)
./l10n_es_partner/gen_src/gen_data_banks.py:47:84: E225 missing whitespace around operator
./l10n_es_partner/gen_src/gen_data_banks.py:48:73: E225 missing whitespace around operator
./l10n_es_partner/gen_src/gen_data_banks.py:48:80: E501 line too long (80 > 79 characters)
./l10n_es_partner/gen_src/gen_data_banks.py:49:74: E225 missing whitespace around operator
./l10n_es_partner/gen_src/gen_data_banks.py:49:80: E501 line too long (80 > 79 characters)
./l10n_es_partner/gen_src/gen_data_banks.py:50:73: E225 missing whitespace around operator
./l10n_es_partner/gen_src/gen_data_banks.py:51:72: E225 missing whitespace around operator
./l10n_es_partner/gen_src/gen_data_banks.py:52:75: E225 missing whitespace around operator
./l10n_es_partner/gen_src/gen_data_banks.py:52:80: E501 line too long (119 > 79 characters)
./l10n_es_partner/gen_src/gen_data_banks.py:53:73: E225 missing whitespace around operator
./l10n_es_partner/gen_src/gen_data_banks.py:53:80: E501 line too long (80 > 79 characters)
./l10n_es_partner/gen_src/gen_data_banks.py:54:72: E225 missing whitespace around operator
./l10n_es_partner/gen_src/gen_data_banks.py:55:74: E225 missing whitespace around operator
./l10n_es_partner/gen_src/gen_data_banks.py:55:80: E501 line too long (81 > 79 characters)
./l10n_es_partner/gen_src/gen_data_banks.py:56:72: E225 missing whitespace around operator
./l10n_es_partner/gen_src/gen_data_banks.py:58:80: E501 line too long (91 > 79 characters)
./l10n_es_partner/gen_src/gen_data_banks.py:59:80: E501 line too long (82 > 79 characters)

Corrígelos, por favor.

[ismaelcj]

Corregido.

[coveralls]

Coverage increased (+0.04%) when pulling 1187828 on factorlibre:7.0 into ff8ea48 on OCA:7.0.

[pedrobaeza]

Ponle por favor extensión .rst para que GitHub lo reconozca, ahora que tenemos esa opción, y habrá que quitar la cabecera de la licencia.

[ismaelcj]

He puesto la extensión .rst y quitado la cabecera.

[rafael-valle]

Buenas,

Hemos añadido el Banco Santander al importador de extractos.

Un saludo.

[coveralls]

Coverage decreased (-0.02%) when pulling bdc7177 on factorlibre:7.0 into ff8ea48 on OCA:7.0.

[pedrobaeza]

Gracias, Rafa, pero hazlo por favor en un PR distinto para no mezclar los temas. ¿Sabes cómo hacerlo?

[rafael-valle]

Buenas Pedro,
Lo cierto es que con github he trabajado poco, tengo que ponerme al día. jajaja

Me imagino que los pasos a seguir es desde OCA/l10n-spain clonarlo, crearme una nueva rama desde la 7.0, subir los cambios y proponeros un merge.¿Me equivoco?.

Un saludo.

[pedrobaeza]

Eso es 😄

[coveralls]

Coverage decreased (-0.02%) when pulling 4d04047 on factorlibre:7.0 into ff8ea48 on OCA:7.0.

[pedrobaeza]

He añadido manualmente en 03e17fb la parte de extractos bancarios, y puesto como autor a Rafa (ya que he visto que él era el que ha hecho el commit).

Ahora después haré lo mismo con la parte del script de bancos y cerraré este PR.

[pedrobaeza]

Y la otra parte del PR la he mergeado aquí: 43b3702. Ismael, tendrás que incluir tu dirección de correo electrónico de FactorLibre para que te reconozca el commit.

Lo he puesto en la v8, ya que el archivo generado es agnóstico a la versión, y lo que se va a traspasar a la próxima versión es este repositorio.

Ahora después generaré el archivo y lo actualizaré en todas las versiones.

Cierro este PR por tanto.

[ismaelcj]

Ok, ¿dónde tengo que poner el correo?

[ismaelcj]

Vale, en el perfil. No he dicho nada.