Page MenuHomePhabricator
Differential D15701

Move server-related Aphlict options to a configuration file
ClosedPublic
Actions

Authored by epriestley on Apr 13 2016, 6:28 PM.
Tags
None
Referenced Files
F16491064: D15701.diff
Mon, Jun 16, 6:42 PM
Unknown Object (File)
Tue, May 27, 11:02 AM
Unknown Object (File)
May 6 2025, 12:16 PM
Unknown Object (File)
May 6 2025, 10:40 AM
Unknown Object (File)
Apr 19 2025, 3:33 AM
Unknown Object (File)
Apr 12 2025, 5:27 AM
Unknown Object (File)
Apr 11 2025, 12:49 AM
Unknown Object (File)
Apr 10 2025, 5:07 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T10697: Move Aphlict server configuration to a separate configuration file
Commits
rPc84dee522bb4: Move server-related Aphlict options to a configuration file
Summary

Ref T10697. This isn't everything but starts generalizing options and moving us toward a cluster-ready state of affairs.

Test Plan

Started server in various configurations, hit most (all?) of the error cases with bad configs, sent test notifications.

Diff Detail

Repository
rP Phabricator
Branch
aphlict3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 11695
Build 14645: Run Core Tests
Build 14644: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 37827.Apr 13 2016, 6:28 PM
epriestley retitled this revision from to Move server-related Aphlict options to a configuration file.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T10697: Move Aphlict server configuration to a separate configuration file.
chad accepted this revision.Apr 13 2016, 6:33 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Apr 13 2016, 6:33 PM
Closed by commit rPc84dee522bb4: Move server-related Aphlict options to a configuration file (authored by epriestley, committed by epriestley). · Explain WhyApr 14 2016, 11:54 AM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in T10751: Make Phabricator Highly Available.Apr 14 2016, 5:50 PM

Revision Contents
Changeset List

PathSize
1 line
conf/
aphlict/
16 lines
18 lines
src/
applications/
aphlict/
management/
197 lines
config/
check/
7 lines
option/
8 lines
docs/
user/
configuration/
45 lines
support/
aphlict/
server/
88 lines

Diff 37827

View Options

.gitignore

Loading...
View Options

conf/aphlict/README

Loading...
View Options

conf/aphlict/aphlict.default.json

Loading...
View Options

src/applications/aphlict/management/PhabricatorAphlictManagementWorkflow.php

Loading...
View Options

src/applications/config/check/PhabricatorExtraConfigSetupCheck.php

Loading...
View Options

src/applications/config/option/PhabricatorNotificationConfigOptions.php

Loading...
View Options

src/docs/user/configuration/notifications.diviner

Loading...
View Options

support/aphlict/server/aphlict_server.js

Loading...
Phabricator · Built by Phacility