melowntech / vts-browser-cpp

VTS Browser C++ library
BSD 2-Clause "Simplified" License
53 stars 19 forks source link

Search Functionality is not working in vts-browser-desktop app #8

Closed VrushalNanavati closed 4 years ago

VrushalNanavati commented 4 years ago

Does the search functionality need any prior setup? It's not working not matter what I query for instance name of the city/region/country. Are there any limitations to this search functionality like only popular cities or places can be queried?

Following is the console log for your reference. 2020-04-02 01:34:39 I3 [46856(main)]: Creating map {mapApiCpp.cpp:vts::Map::Map():137} 2020-04-02 01:34:39 I3 [46856(main)]: Changing mapconfig path to https://cdn.melown.com/mario/store/melown2015/map-config/melown/Melown-Earth-Intergeo-2017/mapConfig.json, without authentication {map.cpp:vts::MapImpl::setMapconfigPath():204} 2020-04-02 01:34:44 I3 [46856(main)]: Mapconfig is available. {map.cpp:vts::MapImpl::prerequisitesCheck():248} 2020-04-02 01:34:44 I3 [46856(main)]: Mapconfig is ready. {map.cpp:vts::MapImpl::prerequisitesCheck():281} 2020-04-02 01:35:39 E2 [46856(chttp:1)]: Exception <Transfer of https://eu-n1.windyty.com/search.php?format=json&addressdetails=1&limit=20&q=new failed: <28, Timeout was reached>.> in download of https://eu-n1.windyty.com/search.php?format=json&addressdetails=1&limit=20&q=new {curl.cpp:vts::anonymous-namespace'::Task::done():197} 2020-04-02 01:35:42 E2 [46856(chttp:1)]: Exception <Transfer of <https://eu-n1.windyty.com/search.php?format=json&addressdetails=1&limit=20&q=new+york> failed: <28, Timeout was reached>.> in download of <https://eu-n1.windyty.com/search.php?format=json&addressdetails=1&limit=20&q=new+york> {curl.cpp:vts::anonymous-namespace'::Task::done():197}

malytomas commented 4 years ago

We have recently changed our search provider. It has been addressed in our javascript browser, and has yet to be fixed in the c++ browser. Usually, changing search provider would require to change the url only, which can be done in the browser options. However, this time, the format has changed a bit too, so it requires slightly more work. We will fix this soon. Anyway, thanks for letting us know.

malytomas commented 4 years ago

This is fixed now. Please open a new issue if you find any further problems.