-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
"Whoops, looks like something went wrong" when importing zkillboard #16
Comments
Have you set your settings correctly, including your API key and chosen system/region/alliance? It won't work without that information. You can get a more detailed stacktrace by looking at the log output, which is saved to app/storage/logs/laravel.php. Try the import again, then paste the last full set of output from the end of that file here. |
Here is the logs:
|
Also, Is it an issue that I am in a wormhole corp? And I selected a Wormhole region? |
You need to post more lines of your log, starting with the line that starts something like this:
It might be WH related, I didn't even know there were wormhole regions... |
|
Okay, so it looks like when it is requesting the zkillboard data it is not getting any results back. It does work with WH systems - for example, here is a request showing all the Wingspan deaths in a WH system: https://zkillboard.com/api/xml/losses/no-attackers/99006319/solarSystemID/31000355/ Do you have an alliance selected? Add this line to the app/controllers/ImportController.php, line 65:
then reload the import/zkillboard page and tell me what it returns. |
Okay - so it seems there are no deaths by Ember Sands members in those systems. Let me patch the import code so it doesn't fail when there are zero kills returned. |
How u know that ;) But ok, ty |
Pushed a patch - if you pull the most recent changes and try again, it should now work. |
I just download the zip and copy paste it, since it didnt work when i used the git clone |
Will that work? |
Yes. |
Still didnt work, get the no attackers |
And now i get Whoops something went wrong when going to settings |
|
You need to remove the debug line I asked you to add earlier. Also I just noticed I forgot to add a new row to the settings table, which I've now pushed. Try again - you will need to re-import the eth_database_setup.sql file. |
I get this error when trying to import the eth_database_setup.sql file #1062 - Duplicate entry '4' for key 'PRIMARY' |
Also, how do i reimport? |
Just delete the settings table first. |
Didnt work |
You used the number 4 twice for home_region_id and home_station_id |
Ah, whoops - fixed now, try again. :) |
It works now! But Since I am in a WH corp, should I add all regions, or what? since there is no real predefined area of operation Edit: It worked w/o an API key somehow... |
I do get an error again when doing the zkillboard import:
|
I think that's due to making too many requests at once. There's a practical limit on how many regions it is appropriate to monitor at once; I never built it to check the whole of New Eden, so you have to decide whereabouts your alliance is most active, and just monitor those places. Re. the API key - it's not used for the import, but you need it if you want accurate T2 invention calculations. |
Oh, I was using my main's api. Not my builders |
One issue I am having now is that I cant update my settings without it reverting |
What do you mean, reverting? You can always just empty the settings table if you're having issues with it, but let me know what is going wrong first. |
The main site works perfectly, the only issue is that I get this message when going to eve-traders-handbook.com/import/zkillboard
My main site is eve-traders-handbook.com. The port I use for MaMp is 80
The text was updated successfully, but these errors were encountered: