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.

Sign up for GitHub

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

Jump to bottom

Change WeatherAgent to use Openweather instead of DarkSky (which is EOL) #2848

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Change WeatherAgent to use Openweather instead of DarkSky (which is EOL) #2848

wants to merge 6 commits into from

Commits on Jul 23, 2020

  1. Remove ForecastIO gem (no longer maintained and DarkSky is EOL as of … 

    …Aug 1, 2020)
    @erebor
    erebor committed Jul 23, 2020
    Configuration menu
    Copy the full SHA
    2944aa9 View commit details
    Browse the repository at this point in the history

  2. Change WeatherAgent to use OpenWeather's free tier and 'DarkSky-match… 

    …ing' onecall API (which only partially matches, but gets a lot right)
    @erebor
    erebor committed Jul 23, 2020
    Configuration menu
    Copy the full SHA
    4d33375 View commit details
    Browse the repository at this point in the history

  3. Chase down and replace more references to DarkSky and replace with Op… 

    …enWeather
    @erebor
    erebor committed Jul 23, 2020
    Configuration menu
    Copy the full SHA
    72b6e23 View commit details
    Browse the repository at this point in the history

  4. Fix spec data for tests to return the right thing for openweather req… 

    …uests
    @erebor
    erebor committed Jul 23, 2020
    Configuration menu
    Copy the full SHA
    e19d8d1 View commit details
    Browse the repository at this point in the history

  5. Put fixture data back to LA weather

    @erebor
    erebor committed Jul 23, 2020
    Configuration menu
    Copy the full SHA
    ab1231b View commit details
    Browse the repository at this point in the history

  6. Make sure test is actually testing the condition we want, and not jus… 

    …t a happy circumstance of the weather data in the fixture
    @erebor
    erebor committed Jul 23, 2020
    Configuration menu
    Copy the full SHA
    2fe3556 View commit details
    Browse the repository at this point in the history