Posted by Dries on February 10, 2009 at 8:51am
| Project: | Drupal.org customizations |
| Version: | 6.x-2.x-dev |
| Component: | User interface |
| Category: | task |
| Priority: | normal |
| Assigned: | jenlampton |
| Status: | needs work |
| Issue tags: | drupal.org redesign, drupal.org redesign qa |
Issue Summary
On the "Get involved" page of the new Drupal.org design (http://drupal.markboultondesign.com/iteration11/getinvolved.html) there is a "Drupal.org activity" block that needs to be implemented.
I created an issue for the cvs_log module to add clean APIs for the CVS related statistics: see #371958: Expose necessary CVS activity statitics for use on d.o.
I created an issue for the project module to add clean APIs for the project related statistics: see #371968: Clean APIs to retrieve project related statistics.
Comments
#1
#2
Moving to the Redesign project. Check here to see who has been assigned to implement this section and the staging server it'll live on during implementation: http://groups.drupal.org/node/37064
Updated prototype (https://infrastructure.drupal.org/drupal.org-style-guide/prototype/getin...)
#3
Moving to section-specific component
#4
drupalorg's quick_stats module does a bit of this, http://drupalcode.org/viewvc/drupal/contributions/modules/drupalorg/bloc.... It needs to be converted to a block, and rolled into the main drupalorg module.
The actual numbers shown can be a bit flexible. For example, Commits this month could be commits per day, or commits in the last 30 days. We should have a concise set of interesting stats that shows how active the Drupal community is.
#5
Subscribing
#6
I added the activity block, it's almost there. I need to look up where to find the number of members on the docs team, but other than that this block is ready to be themed.
This block will need to be added to the Right column region, but does it need to go on any pages other than getting-involved?
#7
I haven't had a chance to review the code yet, but the blocks are reconfigured in drupalorg_update_6500().
#8
Looks good so far. This actually overlaps with the front page a bit too, filling $stats in drupalorg_preprocess_drupalorg_home(). That does some basic caching, too. Maybe we should break that code out into a function and reuse the same cache.
For doc team members, it could be done by checking how many users have the 'documentation maintainer' role. It isn't hugely important to have the exact stats as the prototype, so if it takes more than a few minutes, skip it or put something else interesting and quick in.
#9
@jen, the home page has an activity block of sorts. We could probably just reuse it on the home page (under Develop with Drupal). What do others think?
#10
It looks like there are 296 Documentation maintainers
#11
Query added for docs members.
Block added to update fn.
I created a function to retrieve / update the cached stats that were used on the home page, and added the stats for the getting-involved block to that same function since there was some overlap. I'm a little worried about adding three queries there when they may not be needed (I'm sure the getting involved page won't get nearly as many visits as the front page) but maybe that's alright if they only get run once an hour. If this is a concern, I can break out the additional involved-page stats into a separate stats cache variable, just let me know.
This means we can leave the home page as-is without having to recycle the whole block there.
Here's a list of the stats we need for both pages:
Front page
- cvs accounts
- commits this week
- users
- countries
- languages
Drupal.org activity block (6 queries)
- REUSED: cvs accounts
- REUSED: commits this week (changed from month)
- REUSED: users
- sites
- docs members
- comments
I changed the commits query to be the same in both places (I chose 'this week' over 'this month' - but let me know if month is preferred).
#12
whoopsie.
#13
Looks good. If this becomes a performance problem, the recalculating should be moved to hook_cron() or a drush command, so it never delays a page load.
#14
I made a couple of them lowercase, and removed 'Number of' next to 'people with CVS accounts'
http://redesign.drupal.org/getting-involved
Comments is not showing up though; is this a temporary thing?
#15
I don't think there's any activity on the dev site in the last week. I ran these queries on a more recent db snapshot and got decent numbers. I would love it if someone could double check this for me though.
#16
changing status.
#17
#18
Automatically closed -- issue fixed for 2 weeks with no activity.
#19
I think that this block should be moved to the left most column- atm it is at the bottom of the right most column and pushes the entire area down. Meanwhile, there's plenty of room for it on the left...
#20
Tagging
#21
It is indeed below the fold. At the top of any of the three columns would give it more prominence and it seems like a good lead-in. The middle might also be a good choice (see attached).
#22
I think this is much better, though I still think it makes more sense in the left most column, since there isn't a header there, and it seems like a good lead in for Dries' quote.
#23
Good point, @Alex UA. It would also balance the column heights.
#24