In the tutorial, carry ajax error checking from step 13 forward to other ajax steps #1386
Conversation
… steps The following steps also have an ajax function, but the 'error:' param is gone after facebook#13: facebook#14 facebook#17 facebook#19 facebook#20 This may be superfluous, but it helped me find an error with something I was doing - Namely, in my .json file, I had single line javascript comments ("//") that I copied from the tutorial. I couldn't find the issue on later steps, but was able to see my issue when the error handler complained about an unexpected "/" in my file in step facebook#13.
georgesisco
changed the title
georgesisco
changed the title
georgesisco
changed the title
There was a problem hiding this comment.
If you take a look at line 414 above, you see we have some references to line numbers (looks like {3,16-17,31}) - these allow us to do highlighting of certain lines on the website. Since you're adding lines in the middle here and later, we need to adjust the line numbers so the same thing stays highlighted.
There was a problem hiding this comment.
Ok. I completely missed that. Let me fix that up and push the file again. Thanks for the help!
There was a problem hiding this comment.
@zpao, I've updated the file. Would you take a look? I spent a while trying to figure out how to preview it in it's final form, but I finally just copied each snippet into my editor and grabbed the line numbers from there.
|
This looks great, thanks for updating! |
In the tutorial, carry ajax error checking from step 13 forward to other ajax steps
2 participants
The following steps also have an ajax function, but the 'error:' param
is gone after step 13:
step 14
step 17
step 19
step 20
This may be superfluous, but it helped me find an error with something I
was doing - Namely, in my .json file, I had single line javascript
comments ("//") that I copied from the tutorial. I couldn't find the
issue on later steps, but was able to see my issue when the error
handler complained about an unexpected "/" in my file in step step 13.