Homec4science

Created personal vs. global herald rule distingtion

Authored by awyler <awyler@fb.com> on Jan 14 2012, 00:24.

Description

Created personal vs. global herald rule distingtion

Summary:
A personal rule only has actions targeting the owner. Likewise, only they can
edit the rule. OTOH, a global may affect any target and is editable by anyone.

There are no new action types. Instead, type of the rule modifies the available
targets and the messaging in the ui. This is beneficial because herald rule
adapters don't need to be aware of the difference between emailing the owner of
a personal rule and emailing an arbitrary user.

This diff sets up the logic and ui for creating personal/global rules. All
existing rules have been defaulted to global.

TODO: Filter all existing rules into personal/global
TODO: Create a UI for surfacing (relevant?) global rules.

Test Plan:

  1. Created a personal rule to email myself. Created a dumby revision satisfying

the conditions of that rule. Verified that I recieved a herald email.

  1. Removed my adminship, change the owner of a personal rule. verified that I

couldn't edit the rule.
3.Changed rule type to global. verified that I could edit the rule.

  1. Verified that admins can edit both global and personal rules.

Reviewers: epriestley, jungejason

Reviewed By: epriestley

CC: aran, zizzy

Differential Revision: https://secure.phabricator.com/D1449

Details

Committed
awyler <awyler@fb.com>Jan 19 2012, 20:21
Pushed
aubortJan 31 2017, 17:16
Parents
rPHc4804aa01960: Merge pull request #89 from Koolvin/f6a78452f36454101ab5c8add297657680557091
Branches
Unknown
Tags
Unknown

Event Timeline

awyler <awyler@fb.com> committed rPH6080d7411294: Created personal vs. global herald rule distingtion (authored by awyler <awyler@fb.com>).Jan 19 2012, 20:21