Last modified: 2014-08-07 16:53:35 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T46107, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 44107 - Special:Courses should list Ambassadors for each class
Special:Courses should list Ambassadors for each class
Status: NEW
Product: MediaWiki extensions
Classification: Unclassified
EducationProgram (Other open bugs)
master
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-01-18 14:54 UTC by Sage Ross
Modified: 2014-08-07 16:53 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Sage Ross 2013-01-18 14:54:23 UTC
The table of courses at Special:Courses should include the Ambassadors/volunteers signed up to support each class.

Otherwise, there's no easy way to browse for courses that still need volunteers.
Comment 1 Jeroen De Dauw 2013-01-18 19:30:32 UTC
Just the count? Or a list of actual users? The later might make the table a lot more heavy...
Comment 2 Jeroen De Dauw 2013-01-18 19:32:04 UTC
Alternatively I could also add a filter option that has options

* Any amount of ambassadors
* 5 or more ambassadors
* Less then 5 ambassadors
* No ambassadors
Comment 3 Sage Ross 2013-01-18 19:42:53 UTC
Counts would be fine (and would allow the table to be sortable by total number of ambassadors.

I'm unsure whether it makes sense to count online and campus separately for this, or just show the sum. I'm thinking the distinction doesn't mean much anyway, so the sum would be best. (On a slightly related note, I think it will make sense down the line to combine the Campus and Online Ambassador profiles into one, with fields to designate which role the user is available for, and where, for Campus. But we can think about that more later.)
Comment 4 Jeroen De Dauw 2013-01-18 21:30:24 UTC
Merging the ambassadors into one now makes more sense if you want one count. Right now we cannot efficiently filter or sort by the total count of both sorts of ambs. Just by the count of either type. This will not be a trivial change as it involves db changes and will need some migration work (ie a maintenance script that needs to be run when the code is updated). It thus means we need to be pretty sure this is what we want. Going back would involve similar migration hassle.

Assuming this is what we want, thinking about what exactly we need before the changes are made seems like a good idea. Perhaps making an overview of everything we want to have changed to the ambassador related functionality. Then this can be taken care of during the change to having only one table for ambassadors.
Comment 5 Sage Ross 2013-01-20 17:13:19 UTC
I agree about thinking out what exactly we need before moving forward with merging the ambassadors.

In the short term, having one column each for the Online and Campus counts is probably the best solution.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links