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

Recommended changes to shell-project version to support app.md re-write #16

Open
turbomarc opened this issue Mar 23, 2014 · 0 comments
Open

Comments

@turbomarc
Copy link

The app.md lesson might go more smoothly with a few changes to the shell-project version of global-growth. If the current version is also used for other lessons that would be impacted by these changes, please consider an alternate branch for the proposed app.md version.

*1. Add the following constants and helpers to core.clj:

;; CONSTANTS
(def base-uri "http://api.worldbank.org")
(def list-size 10)

(defn parse-json [str]
  (json/parse-string str true))

*2. Add a line calling out

;; SUPPORTING FUNCTIONS

above the pre-populated supporting functions section.

*3. Include the complete lines for:

(defn view-ind
  [indicator1 indicator2 year]
  (let [inds1 (sorted-indicator-map
                (get-indicator-all indicator1 year :country :value))
        inds2 (sorted-indicator-map
                (get-indicator-all indicator2 year :country :value))]

instead of:

(defn view-ind
  [indicator1 indicator2 year]
  (let [inds1 (<<FILL IN THE BLANK>>)

        inds2 (<<FILL IN THE BLANK>>)]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant