Reviewing merge request #195: Make "most active users" sidebar work again in activity/site index page.

Fixed active user count, now using # of pushes (rather than # of commits which we no longer track the events for?). Also added test coverage for that old User.most_active method.

Requested by: Thomas Kjeldahl Nilsson
Status: Merged
Source repository: thomas-mainline:display-active-users
Target repository: mainline:master
Created at: February 8th, 2012 19:07

Commits that would be merged:

Version 1

Version 1

98e467a
8b9f388

Fixed active users, now using push events as activity metric. Added some test coverage.

Showing

Showing 98e467a-8b9f388

Comments
Include comments entered inline

Pushed new version 1

Thomas Kjeldahl Nilsson | 2012-02-08 19:07:53 UTC

→ State changed from Open to Merged

Sweet-o! Thanks.

Christian Johansen | 2012-02-10 09:58:57 UTC

Add a new comment:

Login or create an account to post a comment

How to apply this merge request to your repository

One way to merge in these changes is to pull them into a local branch for review, verifying the changes and then merge to back to master.

# Check out a new branch for integration
git checkout -b merge-requests/195

# Fetch the merge request into this branch
git pull \
  git://gitorious.org/gitorious/mainline.git \
  refs/merge-requests/195

# Show the commits, assess they are okay
git log --pretty=oneline --abbrev-commit master..merge-requests/195

# To apply the changes to your branch:
git checkout master
git merge merge-requests/195
git push origin master

Reviewing merge request #195: Make "most active users" sidebar work again in activity/site index page.

Commits that would be merged:

Comments Include comments entered inline

Add a new comment:

Gitorious

Comments
Include comments entered inline