{"action":"created","issue":{"url":"https://api.github.com/repos/Semantic-Org/Semantic-UI-React/issues/731","repository_url":"https://api.github.com/repos/Semantic-Org/Semantic-UI-React","labels_url":"https://api.github.com/repos/Semantic-Org/Semantic-UI-React/issues/731/labels{/name}","comments_url":"https://api.github.com/repos/Semantic-Org/Semantic-UI-React/issues/731/comments","events_url":"https://api.github.com/repos/Semantic-Org/Semantic-UI-React/issues/731/events","html_url":"https://github.com/Semantic-Org/Semantic-UI-React/pull/731","id":184908016,"number":731,"title":"perf(props): Remove propTypes from production build","user":{"login":"layershifter","id":14183168,"avatar_url":"https://avatars.githubusercontent.com/u/14183168?v=3","gravatar_id":"","url":"https://api.github.com/users/layershifter","html_url":"https://github.com/layershifter","followers_url":"https://api.github.com/users/layershifter/followers","following_url":"https://api.github.com/users/layershifter/following{/other_user}","gists_url":"https://api.github.com/users/layershifter/gists{/gist_id}","starred_url":"https://api.github.com/users/layershifter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/layershifter/subscriptions","organizations_url":"https://api.github.com/users/layershifter/orgs","repos_url":"https://api.github.com/users/layershifter/repos","events_url":"https://api.github.com/users/layershifter/events{/privacy}","received_events_url":"https://api.github.com/users/layershifter/received_events","type":"User","site_admin":false},"labels":[{"id":226591379,"url":"https://api.github.com/repos/Semantic-Org/Semantic-UI-React/labels/enhancement","name":"enhancement","color":"c5def5","default":true},{"id":274145248,"url":"https://api.github.com/repos/Semantic-Org/Semantic-UI-React/labels/optimization","name":"optimization","color":"fbca04","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":22,"created_at":"2016-10-24T17:44:38Z","updated_at":"2016-11-14T21:57:41Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Semantic-Org/Semantic-UI-React/pulls/731","html_url":"https://github.com/Semantic-Org/Semantic-UI-React/pull/731","diff_url":"https://github.com/Semantic-Org/Semantic-UI-React/pull/731.diff","patch_url":"https://github.com/Semantic-Org/Semantic-UI-React/pull/731.patch"},"body":"# WIP\n\nFor details - #524\n\nThis PR:\n- updates contributing guide;\n- updates `getUnhandledProps` function;\n- adds new sections for common tests;\n- moves definition of handled props to `_meta`.\n\n---\n\n@levithomason I've updated `Rail` component, waiting for review there.\n"},"comment":{"url":"https://api.github.com/repos/Semantic-Org/Semantic-UI-React/issues/comments/260476176","html_url":"https://github.com/Semantic-Org/Semantic-UI-React/pull/731#issuecomment-260476176","issue_url":"https://api.github.com/repos/Semantic-Org/Semantic-UI-React/issues/731","id":260476176,"user":{"login":"levithomason","id":5067638,"avatar_url":"https://avatars.githubusercontent.com/u/5067638?v=3","gravatar_id":"","url":"https://api.github.com/users/levithomason","html_url":"https://github.com/levithomason","followers_url":"https://api.github.com/users/levithomason/followers","following_url":"https://api.github.com/users/levithomason/following{/other_user}","gists_url":"https://api.github.com/users/levithomason/gists{/gist_id}","starred_url":"https://api.github.com/users/levithomason/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/levithomason/subscriptions","organizations_url":"https://api.github.com/users/levithomason/orgs","repos_url":"https://api.github.com/users/levithomason/repos","events_url":"https://api.github.com/users/levithomason/events{/privacy}","received_events_url":"https://api.github.com/users/levithomason/received_events","type":"User","site_admin":false},"created_at":"2016-11-14T21:57:41Z","updated_at":"2016-11-14T21:57:41Z","body":"Using the `unhandledProps` pattern in one of our apps right now, would be great to have the babel plugin for it as well :)  It got me thinking whether or not the plugin should add a `this.getUnhandledProps()` method that just reduces `this.props`.  It seems the primary use case of having `this.constructor.unhandledProps` is to spread unhandled props, so why not provide a method that returns the unhandled props directly?  This could be put behind an option flag if we only want to include the array as the default behavior:\r\n\r\n```\r\n[\"transform-react-handled-props\", {\r\n  \"getUnhandledProps\": true\r\n}]\r\n```"}}
{"actor":{"display_login":"levithomason"}}