Page MenuHomePhabricator

Add "Mailing List" users
ClosedPublic

Authored by epriestley on Jun 2 2015, 6:28 PM.
Tags
None
Referenced Files
F13307117: D13123.id.diff
Sun, Jun 9, 1:47 PM
F13299828: D13123.diff
Fri, Jun 7, 8:29 AM
F13282938: D13123.diff
Sun, Jun 2, 12:37 PM
F13277759: D13123.id31683.diff
Fri, May 31, 12:55 PM
F13277758: D13123.id31682.diff
Fri, May 31, 12:55 PM
F13271477: D13123.diff
Thu, May 30, 12:09 AM
F13271160: D13123.id.diff
Wed, May 29, 5:08 PM
F13264677: D13123.id31682.diff
Mon, May 27, 8:21 PM

Details

Summary

Ref T8387. Adds new mailing list users.

This doesn't migrate anything yet. I also need to update the "Email Addresses" panel to let administrators change the list address.

Test Plan
  • Created and edited a mailing list user.
  • Viewed profile.
  • Viewed People list.
  • Searched for lists / nonlists.
  • Grepped for all uses of getIsDisabled() / getIsSystemAgent() and added relevant corresponding behaviors.
  • Hit the web/api/ssh session blocks.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Add "Mailing List" users.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
  • Don't require "create non-bot users" to see /people/create/ any more.

Hum.

I remember than once i need to login via bot web interface to get cli-token to install certificates.
Since using parameter we can only create "api-xxxxxxxxxxxx" tokens and no "cli-xxxxxxxxxxxxxxx". And since isntallle certificate direclty is depreciated.

btrahan edited edge metadata.
btrahan added inline comments.
src/applications/people/storage/PhabricatorUser.php
118–156

these are nice

This revision is now accepted and ready to land.Jun 3 2015, 11:02 PM
This revision was automatically updated to reflect the committed changes.