Dont send an email when someone views a Phame post
ClosedPublic
Actions

Authored by chad on Aug 25 2017, 1:54 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Apr 25, 3:26 AM

	Unknown Object (File)
	Mon, Apr 22, 2:16 AM

	Unknown Object (File)
	Wed, Apr 10, 9:02 PM

	Unknown Object (File)
	Wed, Apr 10, 5:26 PM

	Unknown Object (File)
	Wed, Apr 10, 5:19 PM

	Unknown Object (File)
	Wed, Apr 10, 5:18 PM

	Unknown Object (File)
	Mon, Apr 8, 12:20 PM

	Unknown Object (File)
	Mon, Apr 8, 11:48 AM

View All 37 Files

Subscribers

Korvin

Details

Reviewers

epriestley
avivey

Commits

rP336fe5cdc525: Dont send an email when someone views a Phame post

Summary

lulz. :(

Test Plan

Load PhamePost, get email. Fix. Reload PhamePost, no email.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad created this revision.Aug 25 2017, 1:54 AM

Herald added a subscriber: Korvin. · View Herald TranscriptAug 25 2017, 1:54 AM

Harbormaster completed remote builds in B18162: Diff 44375.Aug 25 2017, 1:55 AM

chad added a reviewer: avivey.Aug 25 2017, 1:56 AM

I'm assuming this is called for each xaction type?

This revision is now accepted and ready to land.Aug 25 2017, 1:57 AM

It's straight copy/pasta from shouldPublishFeed

Closed by commit rP336fe5cdc525: Dont send an email when someone views a Phame post (authored by Chad Little <chad@phacility.com>). · Explain WhyAug 25 2017, 2:00 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

phame/

editor/

PhamePostEditor.php

6 lines

Diff 44376

View Options

src/applications/phame/editor/PhamePostEditor.php

Dont send an email when someone views a Phame postClosedPublicActions