Include post publish date in phame atom feeds
ClosedPublic
Actions

Authored by mmodell on Mar 5 2018, 6:36 PM.

Details

Reviewers

Samwilson
mmodell

Patch without arc

git checkout -b D997 && curl -L https://phabricator.wikimedia.org/D997?download=true | git apply

Summary

See T188890: Phame blog posts don't have 'published' metadata, only 'updated'

Test Plan

tested on my local phabricator instance.

Diff Detail

Repository

rPHAB Phabricator

Branch

wmf/stable

Lint

Lint OK

Unit

Unit Tests OK

Build Status

Buildable 2819
Build 4714: differential-jessie	Jenkins
Build 4713: arc lint + arc unit

Event Timeline

mmodell created this revision.Mar 5 2018, 6:36 PM

Harbormaster completed remote builds in B2819: Diff 2612.Mar 5 2018, 6:36 PM

mmodell requested review of this revision.Mar 5 2018, 6:36 PM

self accepting because this is safe and tested.

This revision is now accepted and ready to land.Mar 7 2018, 8:42 PM

mmodell closed this revision.Mar 7 2018, 8:45 PM

Revision Contents
Changeset List

Path

Size

src/

applications/

phame/

controller/

blog/

PhameBlogFeedController.php

9 lines

Diff 2612

View Options

src/applications/phame/controller/blog/PhameBlogFeedController.php

Include post publish date in phame atom feedsClosedPublicActions