Code review comment for lp:~halkeye/graphite/graphite

Revision history for this message
Michael Leinartas (mleinartas) wrote :

This looks good, however I saw this merge request too late (sorry) and already merged in basically the same thing in r745 - sorry about that. The difference is that the parameter is called 'jsonp' rather than 'callback', however it works the same way.

r745 didn't have the /graphlot endpoint done as you did so I'm going to make the change to it as you have as well.

Thanks for the contribution

review: Approve

« Back to merge proposal