Shift all the parts up a number.

part7/ElmHub.elm

@@ -1,35 +1,32 @@
 module ElmHub (..) where
 
+import Auth
 import Html exposing (..)
-import Html.Attributes exposing (..)
+import Html.Attributes exposing (class, target, href, property)
 import Html.Events exposing (..)
 import Http
-import Auth
 import Task exposing (Task)
 import Effects exposing (Effects)
 import Json.Decode exposing (Decoder, (:=))
+import Json.Decode.Pipeline exposing (..)
 import Json.Encode
 import Signal exposing (Address)
 
 
-searchFeed : Address String -> String -> Task x Action
-searchFeed address query =
+searchFeed : String -> Task x Action
+searchFeed query =
   let
-    -- See https://developer.github.com/v3/search/#example for how to customize!
     url =
       "https://api.github.com/search/repositories?access_token="
         ++ Auth.token
         ++ "&q="
         ++ query
         ++ "+language:elm&sort=stars&order=desc"
-
-    -- These only talk to JavaScript ports now. They don't
-    -- actually do any actions themselves.
-    task =
-      Signal.send address query
-        |> Task.map (\_ -> DoNothing)
   in
-    Task.onError task (\_ -> Task.succeed DoNothing)
+    performAction
+      (\response -> HandleSearchResponse response)
+      (\error -> HandleSearchError error)
+      (Http.get responseDecoder url)
 
 
 responseDecoder : Decoder (List SearchResult)
@@ -39,16 +36,27 @@ responseDecoder =
 
 searchResultDecoder : Decoder SearchResult
 searchResultDecoder =
-  Json.Decode.object3
-    SearchResult
-    ("id" := Json.Decode.int)
-    ("full_name" := Json.Decode.string)
-    ("stargazers_count" := Json.Decode.int)
+  decode SearchResult
+    |> required "id" Json.Decode.int
+    |> required "full_name" Json.Decode.string
+    |> required "stargazers_count" Json.Decode.int
+
+
+{-| Note: this will be a standard function in Elm 0.17
+-}
+performAction : (a -> b) -> (y -> b) -> Task y a -> Task x b
+performAction successToAction errorToAction task =
+  let
+    successTask =
+      Task.map successToAction task
+  in
+    Task.onError successTask (\err -> Task.succeed (errorToAction err))
 
 
 type alias Model =
   { query : String
   , results : List SearchResult
+  , errorMessage : String
   }
 
 
@@ -67,6 +75,7 @@ initialModel : Model
 initialModel =
   { query = "tutorial"
   , results = []
+  , errorMessage = ""
   }
 
 
@@ -113,26 +122,30 @@ type Action
   = Search
   | SetQuery String
   | DeleteById ResultId
-  | SetResults (List SearchResult)
-  | DoNothing
+  | HandleSearchResponse (List SearchResult)
+  | HandleSearchError Http.Error
 
 
-update : Address String -> Action -> Model -> ( Model, Effects Action )
-update searchAddress action model =
+update : Action -> Model -> ( Model, Effects Action )
+update action model =
   case action of
     Search ->
-      ( model, Effects.task (searchFeed searchAddress model.query) )
+      ( model, Effects.task (searchFeed model.query) )
+
+    HandleSearchResponse response ->
+      -- TODO update the model to incorporate these search results.
+      -- Hint: where would you look to find out the type of `response` here?
+      ( model, Effects.none )
+
+    HandleSearchError error ->
+      -- TODO if decoding failed, store the message in model.errorMessage
+      -- Hint: look for "decode" in the documentation for this union type:
+      -- http://package.elm-lang.org/packages/evancz/elm-http/3.0.0/Http#Error
+      ( model, Effects.none )
 
     SetQuery query ->
       ( { model | query = query }, Effects.none )
 
-    SetResults results ->
-      let
-        newModel =
-          { model | results = results }
-      in
-        ( newModel, Effects.none )
-
     DeleteById idToHide ->
       let
         newResults =
@@ -143,6 +156,3 @@ update searchAddress action model =
           { model | results = newResults }
       in
         ( newModel, Effects.none )
-
-    DoNothing ->
-      ( model, Effects.none )

part7/Main.elm

@@ -5,9 +5,6 @@ import ElmHub exposing (..)
 import Effects exposing (Effects)
 import Task exposing (Task)
 import Html exposing (Html)
-import Signal
-import Json.Encode
-import Json.Decode
 
 
 main : Signal Html
@@ -19,40 +16,12 @@ app : StartApp.App Model
 app =
   StartApp.start
     { view = view
-    , update = update search.address
-    , init = ( initialModel, Effects.task (searchFeed search.address initialModel.query) )
-    , inputs = [ responseActions ]
+    , update = update
+    , init = ( initialModel, Effects.task (searchFeed initialModel.query) )
+    , inputs = []
     }
 
 
 port tasks : Signal (Task Effects.Never ())
 port tasks =
   app.tasks
-
-
-search : Signal.Mailbox String
-search =
-  Signal.mailbox ""
-
-
-port githubSearch : Signal String
-port githubSearch =
-  search.signal
-
-
-responseActions : Signal Action
-responseActions =
-  Signal.map decodeGithubResponse githubResponse
-
-
-decodeGithubResponse : Json.Encode.Value -> Action
-decodeGithubResponse value =
-  case Json.Decode.decodeValue responseDecoder value of
-    Ok results ->
-      SetResults results
-
-    Err _ ->
-      DoNothing
-
-
-port githubResponse : Signal Json.Encode.Value

part7/README.md

@@ -1,4 +1,4 @@
-Part 7
+Part 6
 ======
 
 ## Installation
@@ -16,10 +16,9 @@ to fail; in that case, just run `elm package install` again.)
 elm live Main.elm --open -- --output=elm.js
 ```
 
-## Running Tests
+## References
 
-```bash
-cd test
-elm package install
-elm test TestRunner.elm
-```
+* [HTTP Tasks tutorial](http://elm-lang.org/guide/reactivity#http-tasks)
+* [HTTP Error documentation](http://package.elm-lang.org/packages/evancz/elm-http/3.0.0/Http#Error)
+* [Modules syntax reference](http://elm-lang.org/docs/syntax#modules)
+* [Syntax reference for **case-expressions** and **if-expressions**](http://elm-lang.org/docs/syntax#conditionals)

part7/elm-package.json

@@ -4,10 +4,12 @@
     "repository": "https://github.com/rtfeldman/elm-workshop.git",
     "license": "BSD-3-Clause",
     "source-directories": [
-        ".", ".."
+        ".",
+        ".."
     ],
     "exposed-modules": [],
     "dependencies": {
+        "NoRedInk/elm-decode-pipeline": "1.0.0 <= v < 2.0.0",
         "elm-lang/core": "3.0.0 <= v < 4.0.0",
         "evancz/elm-effects": "2.0.0 <= v < 3.0.0",
         "evancz/elm-html": "4.0.0 <= v < 5.0.0",

part7/index.html

@@ -4,37 +4,23 @@
 <head>
   <meta charset="UTF-8">
   <title>ElmHub</title>
-  <script type="text/javascript" src="github.js"></script>
   <script type="text/javascript" src="elm.js"></script>
+
+  <!-- Uncomment the below line to enable elm-reactor support. -->
+  <!-- <script type="text/javascript" src="/_reactor/debug.js"></script> -->
+
   <link rel="stylesheet" href="style.css">
   <link rel="icon" type="image/png" href="elm-hub.png">
 </head>
 
 <body>
-  <div id="elm-landing-pad"></div>
 </body>
 
 <script type="text/javascript">
-  // documentation: https://github.com/michael/github
-  var github = new Github();
+  var app = Elm.fullscreen(Elm.Main, {});
 
-  var app = Elm.embed(
-    Elm.Main,
-    document.getElementById("elm-landing-pad"),
-    {githubResponse: []});
-
-  function searchGithub(query) {
-    console.log("Searching for", query);
-    var search = github.getSearch(query);
-
-    search.repositories({}, function (err, repositories) {
-      console.log("Got response", repositories);
-
-      // TODO: app.ports.portNameGoesHere.send(repositories);
-    });
-  }
-
-  // TODO app.ports.portNameGoesHere.subscribe(searchGithub);
+  // Uncomment this line and comment out the above to enable elm-reactor support.
+  // var app = Elm.fullscreenDebug("ElmHub", "Main.elm");
 </script>
 
 </html>

part7/test/Tests.elm

@@ -16,13 +16,12 @@ all =
            in
             assertEqual
               (decodeString responseDecoder emptyResponse)
-              (Ok [])
+              ({- TODO: what goes here? -})
     , test "they can decode responses with results in them"
         <| let
             response =
               """{ "items": [
-                { "id": 5, "full_name": "foo", "stargazers_count": 42 },
-                { "id": 3, "full_name": "bar", "stargazers_count": 77 }
+                      /* TODO: dummy JSON goes here */
               ] }"""
            in
             assertEqual