Ah thanks for reminding me of this. Actually this code has a major error.
Will fix tomorrow...
On 5 Mar 2013 21:51, "Martin Filliau" <notifications@github.com> wrote:

> In app/places/collections/POICollection.js:
>
> > -        model: POI
> > +        model: POI,
> > +
> > +        initialize: function(query) {
> > +            this.query = query || {};
> > +        },
> > +
> > +        followUser: function() {
> > +            userPosition.follow(_.bind(this.handle_geolocation_query, this));
> > +        },
> > +
> > +        unfollowUser: function() {
> > +            userPosition.unfollow(_.bind(this.handle_geolocation_query, this));
> > +        },
> > +
> > +        geoFetch: function() { return this.fetch(arguments); },
>
> Could we have this function indented on two lines? Also, might be useful
> to remind to the non ninja JS developer that 'arguments' here corresponds
> to arguments passed to the function...
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ox-it/moxie-js-client/pull/15/files#r3251922>
> .
>