Page MenuHomePhabricator

Add "Mailing List" users
ClosedPublic

Authored by epriestley on Jun 2 2015, 6:28 PM.
Tags
None
Referenced Files
F13476684: D13123.diff
Thu, Jul 18, 4:46 PM
F13472972: D13123.diff
Wed, Jul 17, 12:06 PM
F13447585: D13123.diff
Sat, Jul 13, 3:29 AM
F13438608: D13123.id31683.diff
Thu, Jul 11, 10:39 PM
F13433876: D13123.id.diff
Wed, Jul 10, 6:33 PM
F13423017: D13123.diff
Mon, Jul 8, 11:12 PM
F13418668: D13123.diff
Mon, Jul 8, 1:28 AM
F13400734: D13123.diff
Thu, Jul 4, 12:03 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
Branch
mlist3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 6473
Build 6495: [Placeholder Plan] Wait for 30 Seconds

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.