Well, `api('threads')`


On Mon, Jul 8, 2013 at 3:50 PM, Christopher Van <notifications@github.com>wrote:

> In hearth/media/js/views/comm.js:
>
> > +        $this.parent().siblings('.reply-box').addClass('hidden');
> > +        $this.removeClass('close-reply')
> > +             .addClass('open-reply')
> > +             .html(gettext('Respond'));
> > +
> > +    }).on('keyup', '.reply-text', function(e) {
> > +        var $this = $(this);
> > +        $this.siblings('button.post').
> toggleClass('disabled', $this.val().length < 0);
> > +
> > +    }).on('click', 'button.post', function(e) {
> > +        postNote($(this).siblings('.reply-text'));
> > +    });
> > +
> > +    return function(builder) {
> > +        builder.start('comm/main.html', {
> > +            endpoint: urls.api.url('threads'),
>
> do you need to pass this as a context variable? you can just do
> url('threads') in the template, yeah?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mozilla/commbadge/pull/1/files#r5078575>
> .
>