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.

Sign up for GitHub

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

Jump to bottom

Chapter 5: "we did not copy their container" #35

Closed

jaimeiniesta opened this issue Jun 29, 2015 · 1 comment

jaimeiniesta commented Jun 29, 2015

We again copied the songs from the songs route from the previous chapter and added a new one, the wonderful Daughter from Pearl Jam. We did not copy their container, SongCollection since songs are no longer going to to be listed on their own but next to their band.

But, on the app/routes/band.js snippet just above this paragraph, we can see the SongCollection there, lines 34 and 35:

var songs = SongCollection.create();
songs.pushObjects([blackDog, yellowLedbetter, daughter, pretender]);

(also, a duplicated word: "going to to be listed".

The text was updated successfully, but these errors were encountered:

Collaborator

balinterdi commented Jun 30, 2015

Thank you, Jaime, this was also found a few days ago by another reader (#28) and it will be fixed in the next book release.

balinterdi closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment