Page MenuHomePhabricator
Differential D14459

Only send Phame Post body on new creation
ClosedPublic
Actions

Authored by chad on Nov 10 2015, 8:59 PM.
Tags
None
Referenced Files
F15311895: D14459.diff
Thu, Mar 6, 12:02 PM
Unknown Object (File)
Tue, Feb 18, 10:23 AM
Unknown Object (File)
Fri, Feb 14, 3:51 AM
Unknown Object (File)
Sat, Feb 8, 11:46 PM
Unknown Object (File)
Jan 29 2025, 1:07 PM
Unknown Object (File)
Jan 29 2025, 9:45 AM
Unknown Object (File)
Jan 28 2025, 9:31 PM
Unknown Object (File)
Jan 27 2025, 7:14 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T9360: Unbeta Phame
Commits
Restricted Diffusion Commit
rP9f2fc7f93834: Only send Phame Post body on new creation
Summary

Right now we're attaching the body of every Phame post on each comment, at least restrict it to newly created objects only.

Test Plan

Write a new post, get full email, leave a comment, get less email.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 34966.Nov 10 2015, 8:59 PM
chad retitled this revision from to Only send Phame Post body on new creation.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
Herald added a subscriber: Korvin. · View Herald TranscriptNov 10 2015, 8:59 PM
epriestley accepted this revision.Nov 10 2015, 9:28 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Nov 10 2015, 9:28 PM
Closed by commit rP9f2fc7f93834: Only send Phame Post body on new creation (authored by chad, committed by chad). · Explain WhyNov 10 2015, 9:29 PM
This revision was automatically updated to reflect the committed changes.
chad added a task: T9360: Unbeta Phame.Nov 10 2015, 9:42 PM

Revision Contents
Changeset List

PathSize
src/
applications/
phame/
editor/
5 lines

Diff 34968

View Options

src/applications/phame/editor/PhamePostEditor.php

Loading...
Phabricator · Built by Phacility