skylines icon indicating copy to clipboard operation
skylines copied to clipboard
verified publisher icon skylines-project

[WIP] commands.import.airspace: Skip import of non-available airspaces

Open kerel-fs opened this issue 7 years ago • 1 comments

This commit doesn't implement proper error handling yet, command could result in the deletion of previously downloaded airspaces:

Downloading http://soaringweb.org/Airspace/NL/EHv17_3.txt
Download failed: Server issued an error response.
removing all entries for country_code nl
reading /tmp/nl/nl.openair

Fixes #898.

kerel-fs avatar May 26 '18 10:05 kerel-fs

@kerel-fs this seems like an important fix. Two questions:

  1. Should the server deleted previously downloaded airspaces if the attempt to download a new one fails? Sounds a bit disruptive. For example, you could be trying to update the database and because of one failure now you have none airspaces.
  2. What else do you need in order to move forward with this PR?

mongrelion avatar Oct 15 '20 12:10 mongrelion