{"action":"created","issue":{"url":"https://api.github.com/repos/mlaursen/react-md/issues/117","repository_url":"https://api.github.com/repos/mlaursen/react-md","labels_url":"https://api.github.com/repos/mlaursen/react-md/issues/117/labels{/name}","comments_url":"https://api.github.com/repos/mlaursen/react-md/issues/117/comments","events_url":"https://api.github.com/repos/mlaursen/react-md/issues/117/events","html_url":"https://github.com/mlaursen/react-md/pull/117","id":161248224,"number":117,"title":"Fix issue caused by react-docgen","user":{"login":"jrop","id":1404053,"avatar_url":"https://avatars.githubusercontent.com/u/1404053?v=3","gravatar_id":"","url":"https://api.github.com/users/jrop","html_url":"https://github.com/jrop","followers_url":"https://api.github.com/users/jrop/followers","following_url":"https://api.github.com/users/jrop/following{/other_user}","gists_url":"https://api.github.com/users/jrop/gists{/gist_id}","starred_url":"https://api.github.com/users/jrop/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jrop/subscriptions","organizations_url":"https://api.github.com/users/jrop/orgs","repos_url":"https://api.github.com/users/jrop/repos","events_url":"https://api.github.com/users/jrop/events{/privacy}","received_events_url":"https://api.github.com/users/jrop/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-06-20T17:20:27Z","updated_at":"2016-06-20T18:17:48Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mlaursen/react-md/pulls/117","html_url":"https://github.com/mlaursen/react-md/pull/117","diff_url":"https://github.com/mlaursen/react-md/pull/117.diff","patch_url":"https://github.com/mlaursen/react-md/pull/117.patch"},"body":"Docgen lazy evaluates the expression in `oneOf` in some cases, for example: in Menu.Positions/SelectFields.Positions:\r\n\r\n```js\r\nposition: PropTypes.oneOf(Object.keys(Menu.Positions).map(key => Menu.Positions[key])),\r\n```\r\n\r\nGets compiled by docgen to:\r\n\r\n```js\r\n\"position\": {\r\n    \"type\": \"enum\",\r\n    \"computed\": true,\r\n    \"value\": \"Object.keys(Menu.Positions).map(key => Menu.Positions[key])\"\r\n}\r\n```\r\nOf course, by the time docgen tries to evaluate `Object.keys(Menu.Positions).map(key => Menu.Positions[key])` later, `Menu.Positions` is undefined, causing the documentation site to error when \"Menus\" or \"Select Fields\" is clicked on the documentation site.  When it errors, the page requires to be refreshed completely before it will behave as expected again."},"comment":{"url":"https://api.github.com/repos/mlaursen/react-md/issues/comments/227224710","html_url":"https://github.com/mlaursen/react-md/pull/117#issuecomment-227224710","issue_url":"https://api.github.com/repos/mlaursen/react-md/issues/117","id":227224710,"user":{"login":"mlaursen","id":3920850,"avatar_url":"https://avatars.githubusercontent.com/u/3920850?v=3","gravatar_id":"","url":"https://api.github.com/users/mlaursen","html_url":"https://github.com/mlaursen","followers_url":"https://api.github.com/users/mlaursen/followers","following_url":"https://api.github.com/users/mlaursen/following{/other_user}","gists_url":"https://api.github.com/users/mlaursen/gists{/gist_id}","starred_url":"https://api.github.com/users/mlaursen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mlaursen/subscriptions","organizations_url":"https://api.github.com/users/mlaursen/orgs","repos_url":"https://api.github.com/users/mlaursen/repos","events_url":"https://api.github.com/users/mlaursen/events{/privacy}","received_events_url":"https://api.github.com/users/mlaursen/received_events","type":"User","site_admin":false},"created_at":"2016-06-20T18:17:48Z","updated_at":"2016-06-20T18:17:48Z","body":"Thanks! I thought I fixed it with [this commit](/mlaursen/react-md-docs/commit/4f6480fd68d6e1bfd79471d6f8988f42b0aabeed#diff-bb91cf5047e0d5ad2a8a4eb3421d40a0), but it looks like it still fails on server side rendering. Bleh.\r\n\r\nWould you mind merging this into `release/0.3.x` instead of master?"}}
{"actor":{"display_login":"mlaursen"}}