Page MenuHomePhabricator
Differential D16246

Update Phame documentation to reflect changes to the application
ClosedPublic
Actions

Authored by epriestley on Jul 7 2016, 1:34 AM.
Tags
None
Referenced Files
F13077879: D16246.diff
Tue, Apr 23, 6:32 AM
Unknown Object (File)
Sat, Apr 20, 5:28 AM
Unknown Object (File)
Tue, Apr 9, 10:17 AM
Unknown Object (File)
Mon, Apr 8, 2:22 AM
Unknown Object (File)
Mon, Apr 1, 9:43 AM
Unknown Object (File)
Sat, Mar 30, 10:46 AM
Unknown Object (File)
Sat, Mar 30, 10:46 AM
Unknown Object (File)
Thu, Mar 28, 7:13 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T9895: Update Phame Documentation
T9360: Unbeta Phame
Commits
rPabdb02b51d60: Update Phame documentation to reflect changes to the application
Summary

Ref T9360. Old docs felt a little weird to me (particularly very-old text like "favoring the individual rather than the collective").

Try a simpler tone focused more on use cases and examples?

Test Plan

Read documentation.

Also, viewed a post list and saw monograms.

Diff Detail

Repository
rP Phabricator
Branch
phame1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12957
Build 16541: Run Core Tests
Build 16540: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 39080.Jul 7 2016, 1:34 AM
epriestley retitled this revision from to Update Phame documentation to reflect changes to the application.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T9360: Unbeta Phame.
chad accepted this revision.Jul 7 2016, 1:38 AM
chad edited edge metadata.
chad added inline comments.
src/docs/user/userguide/phame.diviner
116

no post name externally.

This revision is now accepted and ready to land.Jul 7 2016, 1:38 AM
epriestley added a comment.Jul 7 2016, 1:39 AM

There is on posts, but I'll trim that:

http://blog.phacility.com/post/2/pricing_changes_costs_now_stop_increasing_at_50_users/

epriestley updated this revision to Diff 39081.Jul 7 2016, 1:39 AM
epriestley edited edge metadata.
  • Simplify crumb explanation.
chad added a comment.Jul 7 2016, 1:40 AM

there isn't at HEAD, I removed it in the last diff for a cleaner UI (double titles).

epriestley mentioned this in T9895: Update Phame Documentation.Jul 7 2016, 1:40 AM
epriestley added a task: T9895: Update Phame Documentation.
epriestley mentioned this in D14654: Unbeta Phame.
chad added a comment.Jul 7 2016, 1:41 AM

I'll probably give posts the big center text treatment and subtitle.

epriestley added a comment.Jul 7 2016, 1:41 AM

Aaah.

Closed by commit rPabdb02b51d60: Update Phame documentation to reflect changes to the application (authored by epriestley, committed by epriestley). · Explain WhyJul 7 2016, 1:43 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
phame/
query/
1 line
docs/
user/
userguide/
136 lines

Diff 39081

View Options

src/applications/phame/query/PhamePostSearchEngine.php

Loading...
View Options

src/docs/user/userguide/phame.diviner

Loading...
Phabricator · Built by Phacility