Page MenuHomePhabricator
Differential D14618

Add a homepage for Phame
ClosedPublic
Actions

Authored by chad on Nov 30 2015, 10:36 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 17, 9:53 AM
Unknown Object (File)
Tue, Dec 17, 4:18 AM
Unknown Object (File)
Sun, Dec 15, 12:18 PM
Unknown Object (File)
Sun, Dec 1, 10:04 AM
Unknown Object (File)
Sun, Dec 1, 10:04 AM
Unknown Object (File)
Sun, Dec 1, 10:04 AM
Unknown Object (File)
Sun, Dec 1, 10:04 AM
Unknown Object (File)
Sun, Dec 1, 10:03 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T9742: Make Phame navigation hierarchical
Commits
Restricted Diffusion Commit
rP4e6cd90e4164: Add a homepage for Phame
Summary

Sends /phame/ to PhameHomeController, which is all published posts. Still some rough edges to work out for new posts, new blogs, but I think this is the right direction.

Test Plan

go to Phame, see most recent posts, no drafts. click on find posts, see post list, click on find blogs, see blogs.

pasted_file (1×1 px, 178 KB)

Diff Detail

Repository
rP Phabricator
Branch
phame-home-view
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 9173
Build 10841: Run Core Tests
Build 10840: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 35361.Nov 30 2015, 10:36 PM
chad retitled this revision from to Add a homepage for Phame.
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 30 2015, 10:36 PM
chad edited the test plan for this revision. (Show Details)Nov 30 2015, 10:36 PM
chad added a task: T9742: Make Phame navigation hierarchical.
chad added a comment.Nov 30 2015, 10:39 PM

This is a 'base UI' pass, it'll spiffy it up in a future pass.

chad updated this revision to Diff 35363.Nov 30 2015, 11:18 PM
  • Add blog to subtitle on home
epriestley accepted this revision.Nov 30 2015, 11:44 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Nov 30 2015, 11:44 PM
Closed by commit rP4e6cd90e4164: Add a homepage for Phame (authored by chad, committed by chad). · Explain WhyDec 1 2015, 12:26 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
4 lines
applications/
phame/
application/
2 lines
controller/
98 lines
blog/
65 lines
view/
110 lines

Diff 35363

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/phame/application/PhabricatorPhameApplication.php

Loading...
View Options

src/applications/phame/controller/PhameHomeController.php

Loading...
View Options

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

Loading...
View Options

src/applications/phame/view/PhamePostListView.php

Loading...
Phabricator · Built by Phacility