Update Profile Menu diviner
ClosedPublic

Authored by chad on Fri, Feb 3, 10:22 PM.

Details

Summary

Ref T12174, Updates documentation to better match current functionality.

Test Plan

I keep getting exceptions trying to generate diviner locally, not sure what's up.

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
chad created this revision.Fri, Feb 3, 10:22 PM
chad added a comment.Fri, Feb 3, 10:24 PM
BUILDING GRAPH CACHE
Found 6,914 obsolete atoms in graph.
Found 8,932 new atoms in graph.
Propagating changes through the graph.
[2017-02-03 22:23:45] ERROR 8: unserialize(): Error at offset 185 of 907 bytes at [/var/www/html/dev/core/lib/libphutil/src/cache/PhutilDirectoryKeyValueCache.php:58]
arcanist(head=master, ref.master=ade25facfdf2), corgi(head=master, ref.master=aebec8206183), instances(head=master, ref.master=0ddfd7990f6e), libcore(), phabricator(head=diviner-profile-menu, ref.master=42743810e914, ref.diviner-profile-menu=8eb10f48320e), phutil(head=master, ref.master=9d85dfab0f53), services(head=master, ref.master=b5cef1ac31ff)
  #0 unserialize(string) called at [<phutil>/src/cache/PhutilDirectoryKeyValueCache.php:58]
  #1 PhutilDirectoryKeyValueCache::getKeys(array) called at [<phutil>/src/cache/PhutilKeyValueCacheProxy.php:21]
epriestley accepted this revision.Sat, Feb 4, 12:39 AM

Blanket fix that might resolve the unserialize() is to try bin/diviner generate --clean, although that takes quite a while.

(I get some different weirdness running diviner generate locally, maybe a PHP 7 issue? Did you stay on PHP7 or drop back to PHP5? I don't see the unserialize issue, though.)

If that doesn't work you can also just like copy/paste the whole document into a task or comment and look at the preview, which is 95% accurate and good enough to catch stuff like misspelled {icon} or whatever.

src/docs/user/userguide/profile_menu.diviner
130–142

"Let's" -> "Lets": not "Let us you..."

131

"purely" -> "Purely" (beginning of sentence)

This revision is now accepted and ready to land.Sat, Feb 4, 12:39 AM
chad added a comment.Sat, Feb 4, 1:47 AM

it's still building

This revision was automatically updated to reflect the committed changes.