Yes definitely :)


On Wed, Feb 19, 2014 at 11:06 PM, Mihnea Dobrescu-Balaur <
notifications@github.com> wrote:

> In dashboard/js/timeline.js:
>
> > @@ -259,6 +259,14 @@ var TIMELINE_MAPPING = {
> >          },
> >          link: function(e) {
> >              return e.payload.comment.html_url;
> > +        },
> > +        title: function(e) {
> > +          var url = e.payload.comment.html_url;
> > +          var id = url.match(/\/[0-9]+#/g)[0];
> > +          var repo = e.repo.name;
> > +          id = id.substring(1,id.length-1);
> > +          var title = 'Review for pull request <a href="' + url + '">#' + id + '</a><br>';
> > +          return title += '<a href="https://github.com/' + repo + '/pull/'+ id +'/files">View changed files</a>';
>
> there's makeLink in graphs.js, might be better to use that?
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/uberVU/elasticboard/pull/83/files#r9883360>
> .
>



-- 
Andrei Oprea