Differential D13945

Use dateModified for Answer headers in Ponder
ClosedPublic
Actions

Authored by chad on Aug 20 2015, 3:24 PM.

Details

Reviewers

epriestley

Maniphest Tasks

T9226: Answers show create date, not update date

Commits

Restricted Diffusion Commit
rPb35f538bc4cf: Use dateModified for Answer headers in Ponder

Summary

Fixes T9226, shows the update time, not the creation time.

Test Plan

Update an answer, see new date.

Diff Detail

Repository

rP Phabricator

Branch

dateModified

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 7696
Build 8429: [Placeholder Plan] Wait for 30 Seconds
Build 8428: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 33660.Aug 20 2015, 3:24 PM

chad retitled this revision from to Use dateModified for Answer headers in Ponder.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added a reviewer: epriestley.

chad added a task: T9226: Answers show create date, not update date.

Herald added a subscriber: Korvin. · View Herald TranscriptAug 20 2015, 3:24 PM

epriestley accepted this revision.Aug 20 2015, 3:24 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Aug 20 2015, 3:24 PM

Closed by commit rPb35f538bc4cf: Use dateModified for Answer headers in Ponder (authored by chad, committed by chad). · Explain WhyAug 20 2015, 3:25 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

ponder/

view/

PonderAnswerView.php

2 lines

Diff 33660

View Options

src/applications/ponder/view/PonderAnswerView.php

Use dateModified for Answer headers in PonderClosedPublicActions