canonn-science / CAPIv2-Updater

Nodejs based tool to update the Canonn API with planetary and stellar data from EDSM
GNU General Public License v3.0
1 stars 0 forks source link

updateSystems doesn't seem to be actually pushing data #11

Closed derrickmehaffy closed 5 years ago

derrickmehaffy commented 5 years ago

Tested with both build and dev commands using the newly updated staging server, it appears that the updater is getting data from EDSM but isn't pushing the data to the API.

If the updater is trying to send nested PUT requests that would be the reason, each system will need to be updated in a single PUT request.

Based on the following code: https://github.com/canonn-science/CAPIv2-Updater/blob/115f1f725cb1a1976228638d2aad99ac5716512d/src/api/canonn.js#L153-L170

It looks like it might be trying to push a payload then querying each system one by one afterwords.

coldwhitelight commented 5 years ago

Fixed with #12