sigdeletras / Spanish_Inspire_Catastral_Downloader

A QGIS plugin to download cadastral data from INSPIRE Services of Spanish Cadastral Cartography
GNU General Public License v3.0
23 stars 11 forks source link

Sustitución de listados estáticos por peticiones REST #24

Closed lgarciademarina closed 1 year ago

lgarciademarina commented 1 year ago

Se sustituyen las listas estáticas de provincias y municipios por llamadas a servicios REST, de esta forma estarán siempre actualizadas. Se hace búsqueda del ZIP del municipio dentro del xml ATOM de la provincia correspondiente mediante el código completo del municipio, de esta forma no hay problemas con los caracteres especiales en el nombre del municipio. Se añaden comprobaciones sobre la url de descarga y si el conujunto de datos existe en la carpeta de destino (para no volver a procesar).

sigdeletras commented 1 year ago

Hola @lgarciademarina ¿qué tal?

A partir de tu PR he credo una rama en el repositorio lgarciademarina-master donde he integrado tu imporatnte aporte. Gracias de nuevo

He aprovechado para revisar el complemento, arreglar otros fallos menores y limpiar un poco. Esto constituirá la V2.0.

Antes de hacer merge de la rama y publicarlo en el repositorio oficial te quería pedir el favor de si podias testear los cambios y comporbar que todo está Ok. Te lo agradecería mucho.

Un saludo

lgarciademarina commented 1 year ago

Hola Patricio, Esta semana le echo un ojo a todo y te digo. Muchas gracias.

El jue, 12 oct 2023, 20:06, Patricio Soriano @.***> escribió:

Hola @lgarciademarina https://github.com/lgarciademarina ¿qué tal?

A partir de tu PR he credo una rama en el repositorio lgarciademarina:master https://github.com/lgarciademarina/Spanish_Inspire_Catastral_Downloader/tree/master donde he integrado tu imporatnte aporte. Gracias de nuevo

He aprovechado para revisar el complemento, arreglar otros fallos menores y limpiar un poco. Esto constituirá la V2.0.

  • Integrado PR de Laura García de Marina https://github.com/lgarciademarina para integrar los listados de provincias y municipios desde los servicios de catastro
  • Se separa los pasos de descarga y añadir capas al proyecto
  • El complemento ya es solo compatible con versiones 3.* de QGIS
  • Añadidos más mensajes de aviso al usuario
  • No es ncesario convertir a geojson los GML. Se cargan ya directamente.
  • Revisiones y mejoras en el estilo de código

Antes de hacer merge de la rama y publicarlo en el repositorio oficial te quería pedir el favor de si podias testear los cambios y comporbar que todo está Ok. Te lo agradecería mucho.

Un saludo

— Reply to this email directly, view it on GitHub https://github.com/sigdeletras/Spanish_Inspire_Catastral_Downloader/pull/24#issuecomment-1760116407, or unsubscribe https://github.com/notifications/unsubscribe-auth/AYFSKD7VRMD4NDF7FDODBIDX7AWQ5AVCNFSM6AAAAAA2PMR33SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRQGEYTMNBQG4 . You are receiving this because you were mentioned.Message ID: <sigdeletras/Spanish_Inspire_Catastral_Downloader/pull/24/c1760116407@ github.com>