Table displayed in Special:ManageNewsletter when the user has nothing to manage
Closed, DeclinedPublic
Actions

Assigned To

Authored By

	Tinaj1234
	Oct 29 2015, 2:08 PM

Description

Two choices:

Don't display the table
Keep the table and fill the 'Action' column with 'None' or 'No permissions'

Details

	Subject	Repo	Branch	Lines +/-
	Sort newsletter names and few minor changes on ManageNewsletter	mediawiki/extensions/Newsletter	master	+7 -3

Customize query in gerrit

Related Objects

Mentioned In: T110327: Add user rights for Newsletter extension

Event Timeline

Tinaj1234 created this task.Oct 29 2015, 2:08 PM

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

Tinaj1234 updated the task description. (Show Details)

Tinaj1234 added a project: MediaWiki-extensions-Newsletter.

Tinaj1234 added subscribers: Tinaj1234, Qgil, 01tonythomas, Glaisher.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 29 2015, 2:08 PM

Not displaying the table is better. If a user has permissions to manage only one newsletter, does she see one newsletter or all of them? Think also in the day when there will be hundreds of newsletter available.

Change 249996 had a related patch set uploaded (by Tinaj1234):
Sort newsletter names and few minor changes on ManageNewsletter

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

gerritbot added a project: Patch-For-Review.Oct 30 2015, 1:14 PM

The patch submitted for this task makes sure that the entries listed in the table are just those newsletters which can be managed by the logged in user. Should a filter for the table be added as well ?

Restricted Application added a subscriber: StudiesWorld. · View Herald TranscriptNov 12 2015, 7:00 AM

Tinaj1234 mentioned this in T110327: Add user rights for Newsletter extension.Nov 12 2015, 7:39 AM

Qgil assigned this task to Tinaj1234.Nov 12 2015, 11:23 AM

Qgil triaged this task as Medium priority.

Qgil moved this task from Backlog to Feature complete on the MediaWiki-extensions-Newsletter board.

Qgil set Security to None.

Change 249996 abandoned by 01tonythomas:
Sort newsletter names and few minor changes on ManageNewsletter

Reason:
We just removed this file NewsletterManageTablePager.php!