This creates http://drupal.org/metrics. The implementation may change to use Sampler.

CommentFileSizeAuthor
#1 1599100-v1.patch5.06 KBjhodgdon

Comments

jhodgdon’s picture

Status: Active » Needs review
StatusFileSize
new5.06 KB

Here's a patch that gets the metrics page working, except for the commits stuff, which needs to wait. (Issue for that is: #1683846: Turn on commit-related parts of drupalorg_metrics).

I'll double-check applying this patch and report back...

jhodgdon’s picture

This patch seems to work fine... There is one bluecheese/formatting change: the links at the top of the /metrics page are in-line on drupal.org/ and a vertical list on the dev D7 site.

luukyb’s picture

Maybe related to another issue but after aplying the patch, activated the module and executed the function drupalorg_metrics_record_stats, I'm getting a javascript error on the metrics page :
jQuery.plot is not a function

jhodgdon’s picture

It could be that there are dependencies of this module that are on drupal.org but that you do not have. I don't know what they would be, but the patch does work fine on a drupal.org d7 dev site... It looks like you are missing a jQuery library...

luukyb’s picture

Yeah the flot librairy was missing. Works fine with it.

jhodgdon’s picture

The flot module is already a dependency of this module, so I don't think we need to change the module or the patch in any way.

drumm’s picture

Status: Needs review » Fixed

Committed.

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit 9fe752b on 7.x-3.x, 7.x-3.x-dev by drumm:
    [#1599100] --author="jhodgdon <jhodgdon@155601.no-reply.drupal.org>"