Sorting of newsletters
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Qgil
	Jul 31 2015, 8:48 AM

Description

Special:Newsletters provides a list of newsletters -- see http://newsletter-test.wmflabs.org/wiki/Special:Newsletters

Currently the sorting is... unclear. This is not very important when we have only four newsletters for testing purposes, but in a real scenario sorting matters, and a lot. Maybe one day we will have categories of newsletters, different ways of sorting them and what not. But let's just aim to have a logical first step good enough for a first release.

Proposal:

Newsletters the user has subscribed to go to the top, followed by the rest.
Within subscribed / non-subscribed newsletters, those with more subscribers go to the top.

This sorting doesn't need live refresh when the user changes their subscriptions. Whenever the user reloads or visits the page again, the table wll be updated. Live refresh would create jumps that might confuse the user. Imagine that you are subscribed to a newsletter with 5 subscribers, click unsubscribe kind of for testing purposes, and then the entry "disappears", being dumped down the list.

Details

	Subject	Repo	Branch	Lines +/-
	Add filters to customize the search of newsletters	mediawiki/extensions/Newsletter	master	+89 -6
	Correct sorting of newsletters in Special: Newsletters	mediawiki/extensions/Newsletter	master	+2 -1

Customize query in gerrit

Related Objects
Search...

Status	Assigned	Task
Duplicate	Qgil	T125545 Phabricator Q&A session for Community Liaisons
Resolved	Qgil	T116025 Goal: Align Community Liaison and Developer Relations project management practices
Resolved	Qgil	T119387 Community Liaison and Developer Relation quarterly goals for January - March 2016
Declined	None	T104131 Exporting existing newsletter to the Newsletter extension
Resolved	Addshore	T110170 Goal: Deploy Newsletter extension in Wikimedia
Resolved	Qgil	T110642 Implement all the features required for running the Newsletter extension in Wikimedia
Resolved	Tinaj1234	T107557 Sorting of newsletters

Event Timeline

Qgil created this task.Jul 31 2015, 8:48 AM

Qgil raised the priority of this task from to Needs Triage.

Qgil updated the task description. (Show Details)

Qgil added a project: MediaWiki-extensions-Newsletter.

Qgil subscribed.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 31 2015, 8:48 AM

Qgil triaged this task as Medium priority.Aug 28 2015, 11:27 AM

Qgil moved this task from Backlog to Feature complete on the MediaWiki-extensions-Newsletter board.Aug 28 2015, 11:54 AM

Change 235420 had a related patch set uploaded (by Tinaj1234):
Correct sorting of newsletters in Special: Newsletters

https://gerrit.wikimedia.org/r/235420

gerritbot added a project: Patch-For-Review.Sep 2 2015, 8:52 AM

Newsletters the user has subscribed to go to the top, followed by the rest.

The current patch accomplishes this.

Within subscribed / non-subscribed newsletters, those with more subscribers go to the top.

@Addshore I didn't find a way to incorporate two ways of sorting in one table using TablePager. As far as I understood, getDefaultSort() is called once in the constructor and that's it, you can choose one field name to set the sort-order. Any ideas ?

Change 235420 merged by jenkins-bot:
Correct sorting of newsletters in Special: Newsletters

https://gerrit.wikimedia.org/r/235420

Tinaj1234 mentioned this in rMEXT640f75471dd3: Updated mediawiki/extensions Project: mediawiki/extensions/Newsletter….Sep 2 2015, 9:07 AM

Tinaj1234 mentioned this in rENLT0b0262e63eea: Correct sorting of newsletters in Special: Newsletters.

In T107557#1596200, @Tinaj1234 wrote:

@Addshore I didn't find a way to incorporate two ways of sorting in one table using TablePager. As far as I understood, getDefaultSort() is called once in the constructor and that's it, you can choose one field name to set the sort-order. Any ideas ?

What about this:

a first list/table at the top with the newsletters you are subscribed to, followed by
another table/list with all the rest of newsletters

Qgil assigned this task to Tinaj1234.Oct 1 2015, 11:14 AM

<Glaisher> two tables..? Why two tables?
<Glaisher> Can't we just simply add a filter?
<Glaisher> two tables would look ugly on one page, I think
<Glaisher> especially, with pagination

We did not think about the pagination. A filter sounds more sensible.

Yes, indeed.

Change 247843 had a related patch set uploaded (by Tinaj1234):
Add filters to customize the search of newsletters

https://gerrit.wikimedia.org/r/247843

Glaisher added a parent task: T110642: Implement all the features required for running the Newsletter extension in Wikimedia.Oct 23 2015, 11:33 AM

• MZMcBride subscribed.Oct 24 2015, 6:03 PM

Change 247843 merged by jenkins-bot:
Add filters to customize the search of newsletters

https://gerrit.wikimedia.org/r/247843

Glaisher closed this task as Resolved.Nov 16 2015, 4:49 PM

Glaisher subscribed.