Differential D13138

Allow administrators to configure "Email Settings" settings for lists
ClosedPublic
Actions

Authored by epriestley on Jun 3 2015, 2:28 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Jan 25, 11:17 AM

	Unknown Object (File)
	Fri, Jan 24, 5:52 AM

	Unknown Object (File)
	Fri, Jan 24, 5:52 AM

	Unknown Object (File)
	Fri, Jan 24, 5:52 AM

	Unknown Object (File)
	Tue, Jan 21, 4:08 PM

	Unknown Object (File)
	Tue, Jan 21, 11:44 AM

	Unknown Object (File)
	Tue, Jan 21, 10:06 AM

	Unknown Object (File)
	Fri, Jan 17, 7:05 PM

Subscribers

Details

Reviewers

Maniphest Tasks

T8387: Convert Mailing Lists into special users, similar to bot users

Commits

Restricted Diffusion Commit
rPa4d874287ff6: Allow administrators to configure "Email Settings" settings for lists

Summary

Fixes T8387. This completes conversion of lists into users.

These settings allow administrators to reduce the amount of mail delivered to lists, ahead of T5791.

Test Plan

Screen Shot 2015-06-03 at 7.26.49 AM.png (1×2 px, 425 KB)

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 31729.Jun 3 2015, 2:28 PM

epriestley retitled this revision from to Allow administrators to configure "Email Settings" settings for lists.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

epriestley added a task: T8387: Convert Mailing Lists into special users, similar to bot users.

Herald added a subscriber: epriestley. · View Herald TranscriptJun 3 2015, 2:28 PM

UI seems a little strange for the "self actions" thing -- mailing list users can't take actions, right? -- but probably whatevs.

This revision is now accepted and ready to land.Jun 3 2015, 11:14 PM

Yeah, we could hide "Self Actions" (and probably "don't send me email", since it seems pointless and you can disable the user instead) but I was also on the whatevs train here.

Closed by commit rPa4d874287ff6: Allow administrators to configure "Email Settings" settings for lists (authored by epriestley, committed by epriestley). · Explain WhyJun 4 2015, 1:43 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

settings/

panel/

PhabricatorEmailPreferencesSettingsPanel.php

13 lines

Diff 31766

src/applications/settings/panel/PhabricatorEmailPreferencesSettingsPanel.php

Loading...