Differential D14914

Update some PhamePost transactions
ClosedPublic
Actions

Authored by chad on Dec 30 2015, 8:08 PM.

Details

Reviewers

epriestley

Maniphest Tasks

T9360: Unbeta Phame

Commits

Restricted Diffusion Commit
rP5ea5b0c41c61: Update some PhamePost transactions

Summary

Cleans up some language, colors, etc.

Test Plan

Write lots of new posts, hide them, edit them, check history.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad updated this revision to Diff 36039.Dec 30 2015, 8:08 PM

chad retitled this revision from to Update some PhamePost transactions.

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 TranscriptDec 30 2015, 8:08 PM

chad added inline comments.Dec 30 2015, 8:09 PM

src/applications/phame/storage/PhamePostTransaction.php
219	not sure how to get the body to show up on CREATE transactions, but at least it doesn't show on edits now.

chad added a task: T9360: Unbeta Phame.Dec 30 2015, 8:31 PM

epriestley accepted this revision.Dec 30 2015, 8:54 PM

epriestley edited edge metadata.

epriestley added inline comments.

src/applications/phame/storage/PhamePostTransaction.php
219	This should have the same effect anyway.

This revision is now accepted and ready to land.Dec 30 2015, 8:54 PM

Closed by commit rP5ea5b0c41c61: Update some PhamePost transactions (authored by chad, committed by chad). · Explain WhyDec 30 2015, 8:57 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

phame/

storage/

PhamePostTransaction.php

63 lines

Diff 36043

View Options

src/applications/phame/storage/PhamePostTransaction.php

Update some PhamePost transactionsClosedPublicActions