search

vltansky / IkaEasy

Chrome extention for Ikariam. WARNING: this repo is not maintained anymore. the new IkaEasy (v3) moved to a private repo
http://ikariam-easy.com/
MIT License
5 stars 2 forks source link

Fix building calc #36

Closed theotonchev closed 7 years ago

theotonchev commented 7 years ago

IkaEasy currently does not calculate the resources needed for a building correctly -- the red/green indicators next to the buildings are off. The reason is that it does not take into account the completed researches.

The root cause was eventually tracked to the fact that Ikariam has apparently slightly modified the returned JSON that is used to determine the research levels. The change adapts the code to the new model.

In addition, the AJAX requests were not going through if the browser connection to the server was https -- the AJAX requests were failing with a "Mixed Content" error as they were using http. Another commit in this set fixes that problem as well.

vltansky commented 7 years ago

Thanks for commit! Will update extention soon!

theotonchev commented 7 years ago

Hi Vlad,

Happy New Year!

I was going to write you to let you know, but started working on another issue as well (pirate point conversion dialog gets corrupted when the plugin is enabled) and was wondering whether I would be able to include it here as well to minimize the number of updates. Given time availability, however, it does not appear that would happen very quickly, so I will make a separate commit for that when done. So please update when convenient.

Have a successful 2017!

On Mon, Jan 2, 2017 at 10:19 PM, Vlad Tansky notifications@github.com wrote:

Merged #36 https://github.com/swat-web/IkaEasy/pull/36.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/swat-web/IkaEasy/pull/36#event-909235079, or mute the thread https://github.com/notifications/unsubscribe-auth/AXPoG5sPdpKQk6GI6KVZxgbmNERsHFsVks5rOVvCgaJpZM4LY5XA .