Fix `!unsubscribe` in Differential
ClosedPublic
Actions

Authored by epriestley on Jul 22 2014, 2:28 PM.

Details

Reviewers

btrahan
chad

Maniphest Tasks

T5682: !unsubscribe not working in Differential because of missing setActor()

Commits

Restricted Diffusion Commit
rP1649cf94f41e: Fix `!unsubscribe` in Differential

Summary

Fixes T5682.

Test Plan

Will test...

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 24068.Jul 22 2014, 2:28 PM

epriestley retitled this revision from to Fix `!unsubscribe` in Differential.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added reviewers: chad, btrahan.

epriestley added a task: T5682: !unsubscribe not working in Differential because of missing setActor().

Herald added a subscriber: epriestley. · View Herald TranscriptJul 22 2014, 2:28 PM

Harbormaster completed remote builds in B1833: Diff 24068.Jul 22 2014, 2:29 PM

I tested this locally by piping mail into mail_handler.php and its seems to work correctly.

chad accepted this revision.Jul 22 2014, 2:31 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Jul 22 2014, 2:31 PM

Closed by commit rP1649cf94f41e (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

mail/

DifferentialReplyHandler.php

1 line

Diff 24069

View Options

src/applications/differential/mail/DifferentialReplyHandler.php

Fix `!unsubscribe` in DifferentialClosedPublicActions