Description
Context, like, "before the intl:..." would be helpful. Also, a comma is required to separate the blocks, so if it is going to be before the intl:, then a comma is required after the closing brace. I realize that this is probably newbie stuff on my part, but this module is not consistent with the style of the other modules, and if people are copy-pasting fast, trying to keep up or catch up, it would be helpful to be consistent.
Also, after I did figure out the syntax changes required to make the supplied code fit in with app/routes/application.js, I got the following error output in the ember s terminal:
/Users/jere7054/Documents/ambitious2/ambitious-arcgis-app/app/routes/application.js
14:7 error 'debug' is not defined no-undef
17:7 error 'debug' is not defined no-undef
✖ 2 problems (2 errors, 0 warnings)
Similarly, it wasn't clear precisely where the navbar stuff was supposed to go. I guessed and got it wrong. I only figured out where it was really supposed to go by cheating and looking at our repo. Again, I don't know if this is newbie stuff that will be obvious to the actual students in the workshop, but it was a stumbling block for me.