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

	F18989906: D18471.diff
	Tue, Nov 18, 4:29 PM

	F18770922: D18471.diff
	Oct 8 2025, 4:11 PM

	F18766843: D18471.id44376.diff
	Oct 7 2025, 6:52 PM

	F18096694: D18471.id44376.diff
	Aug 8 2025, 8:39 AM

	F18093538: D18471.id.diff
	Aug 7 2025, 7:12 PM

	F18093525: D18471.id44375.diff
	Aug 7 2025, 7:03 PM

	F18090660: D18471.diff
	Aug 6 2025, 4:45 PM

	Unknown Object (File)
	May 7 2025, 2:02 PM

View All 99 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