EggsML icon indicating copy to clipboard operation
EggsML copied to clipboard

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

Open Sword-Smith opened this issue 6 years ago • 3 comments

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 avatar Sep 25 '19 19:09 Sword-Smith

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 avatar Nov 15 '19 06:11 Sword-Smith

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?

Sword-Smith avatar Nov 15 '19 06:11 Sword-Smith

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

nqpz avatar Nov 18 '19 20:11 nqpz