I wasn't able to get to this today (need sleep now). Maybe it would work better to take that on after #2 is merged?

Hope my latest commits are improvements :) It was fun knocking my additions out of main.js

-Tim

On Dec 18, 2013, at 12:40 AM, Joe Fleming <notifications@github.com> wrote:

> In public/js/App.js:
> 
> > @@ -42,7 +43,17 @@ App.Piece = Backbone.Model.extend({
> >      this.firebase = 'https://holiday-js-hackathon-2013.firebaseio.com/' + uuid4.toString();
> >      // this.listenTo('add', this, initPiece);
> >    }
> > -});;App.PieceView = Backbone.View.extend({
> > +});;/* TODO: move UUID code to main.js */
> > +App.Players = Backbone.Firebase.Collection.extend({
> > +  model: App.Player,
> > +  // firebase: 'https://holiday-js-hackathon-2013.firebaseio.com/',
> > +  initialize: function() {
> > +    var uuid4       = UUIDjs.create();
> > +    this.firebase = 'https://holiday-js-hackathon-2013.firebaseio.com/' + uuid4.toString();
> If you want to fix this right now, take a look at the Pieces collection in #2
> 
> The gist is that this should look like this.firebase = App.firebaseUrl + App.gameInstance + 'players/';
> 
> —
> Reply to this email directly or view it on GitHub.
>