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

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

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

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

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

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

	Unknown Object (File)
	Apr 30 2025, 6:51 AM

	Unknown Object (File)
	Mar 4 2025, 1:08 AM

	Unknown Object (File)
	Feb 21 2025, 2:06 PM

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

Branch

phame-email (branched from master)

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 18162
Build 24413: Run Core Tests
Build 24412: arc lint + arc unit

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 44375

View Options

src/applications/phame/editor/PhamePostEditor.php

Dont send an email when someone views a Phame postClosedPublicActions