Page MenuHomePhabricator

Pholio notifications are not configurable in /settings/panel/emailpreferences/
Closed, ResolvedPublic

Description

I just noticed this after receiving an email about a comment on a Mock while testing having all email notifications off (and use the site notifications instead).

Revisions and Commits

Event Timeline

swisspol raised the priority of this task from to Needs Triage.
swisspol updated the task description. (Show Details)
swisspol added a subscriber: swisspol.
chad triaged this task as Normal priority.Jun 17 2014, 1:09 AM
chad added a project: Pholio.
chad added subscribers: chad, epriestley.

@epriestley - this seems "easy" to me, is the current infrastructure ok to copy/paste (maniphest, differential) or is there something more modern. It seems like we offer these features on 2 applications.

I think this is fairly straightforward for everything except Diffusion (until T4896), but probably needs to be generalized a bit since if every app copy/pasted the current stuff we'd have like 100 checkboxes on that page.

we could write some javascript (check all, uncheck all). just going to do pholio for now, as most of our other apps aren't very 'chatty'.