From 06e406c68cdbe8f638726fb7ef2f99946b1e64e7 Mon Sep 17 00:00:00 2001 From: dcsmith926 Date: Wed, 25 Nov 2015 18:38:19 -0500 Subject: [PATCH] use of ".map()" should be ".forEach()" In "seed/challenges/json-apis-and-ajax.json", the use of ".map()" in the 5th and 6th challenges doesn't make sense -- we're not creating a new array by modifying each element of the response, and in challenge 4, the learner was told to use ".forEach()". Seeing ".map()" instead for no apparent reason would likely be a source of confusion. --- challenges/json-apis-and-ajax.json | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/challenges/json-apis-and-ajax.json b/challenges/json-apis-and-ajax.json index a03fa1b9a8f..147fc9abbd4 100644 --- a/challenges/json-apis-and-ajax.json +++ b/challenges/json-apis-and-ajax.json @@ -239,7 +239,7 @@ "", " var html = \"\";", "", - " json.map(function(val) {", + " json.forEach(function(val) {", "", " html += \"
\";", "", @@ -310,7 +310,7 @@ " ", " // Only change code above this line.", "", - " json.map(function(val){", + " json.forEach(function(val) {", "", " html += \"
\"", "",