search

athas / EggsML

A fully fledged and highly scalable lunch management system for the modern enterprise
http://eggsml.dk
GNU Affero General Public License v3.0
30 stars 10 forks source link

Almanak burde anvende yr.no's API i stedet for openweathermaps #161

Open Sword-Smith opened 5 years ago

Sword-Smith commented 5 years ago

vejrudsigt bruger yr.no's API, og der har vi implementeret fritekstsøgning på lokationer. Hvis vi bruger samme API til almanak så kan man skrive concieggs: almanak Boston i stedet for, som nu, at skulle skrive concieggs: almanak Boston,USA.

Og vejrudsigt indlæser allerede solopgang-/-nedgangstiderne, så man kan bare se, hvordan det gøres det og så gøre det efter.

Sword-Smith commented 5 years ago

yr.no har vejr for langt flere steder end openweathermaps. Jeg downloadede en liste over openweathermaps steder, og der kunne jeg f.eks. hverken finde Flekke i Sunnfjord (i Norge) eller Hellerup, begge steder findes på yr.no. Så løsningen er nok at udskille den del af koden (get_url_from_search_string), der finder API-URL'en på yr.no, og som ligger i concieggs/eggspi/vejr og så lade denne blive delt mellem vejrudsigt og almanak.

Sword-Smith commented 5 years ago

Alternativt kunne vi lægge ovenstående liste ind et sted, hvor concieggs kan tilgå det, og så kan man regexe/json-søge sig rundt i den liste og bruge det til at finde et ID, som man så kan bruge til at slå vejret op. Det burde vi faktisk gøre, da vejr bruger openweathermaps, og den kommando nok ikke bliver ændret til at bruge yr.no. Måske kan @nqpz hjælpe med at lægge listen ind et fornuftigt sted?

nqpz commented 5 years ago

Jeg synes at det er en meget stor liste. Jeg vil hellere dele vejr-funktionen med vejrudsigt og almanak.