Skip to content
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

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

Open
Sword-Smith opened this issue Sep 25, 2019 · 3 comments
Open

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

Sword-Smith opened this issue Sep 25, 2019 · 3 comments
Assignees

Comments

@Sword-Smith
Copy link
Collaborator

Sword-Smith commented Sep 25, 2019

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
Copy link
Collaborator Author

Sword-Smith commented Nov 15, 2019

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
Copy link
Collaborator Author

Sword-Smith commented Nov 15, 2019

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
Copy link
Collaborator

nqpz commented Nov 18, 2019

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants