{"action":"created","issue":{"url":"https://api.github.com/repos/facebook/react/issues/1138","repository_url":"https://api.github.com/repos/facebook/react","labels_url":"https://api.github.com/repos/facebook/react/issues/1138/labels{/name}","comments_url":"https://api.github.com/repos/facebook/react/issues/1138/comments","events_url":"https://api.github.com/repos/facebook/react/issues/1138/events","html_url":"https://github.com/facebook/react/issues/1138","id":27946093,"number":1138,"title":"\"Mixed in\" callback for mixins","user":{"login":"andreypopp","id":30594,"avatar_url":"https://avatars.githubusercontent.com/u/30594?v=3","gravatar_id":"","url":"https://api.github.com/users/andreypopp","html_url":"https://github.com/andreypopp","followers_url":"https://api.github.com/users/andreypopp/followers","following_url":"https://api.github.com/users/andreypopp/following{/other_user}","gists_url":"https://api.github.com/users/andreypopp/gists{/gist_id}","starred_url":"https://api.github.com/users/andreypopp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andreypopp/subscriptions","organizations_url":"https://api.github.com/users/andreypopp/orgs","repos_url":"https://api.github.com/users/andreypopp/repos","events_url":"https://api.github.com/users/andreypopp/events{/privacy}","received_events_url":"https://api.github.com/users/andreypopp/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/facebook/react/labels/Component:%20Core","name":"Component: Core","color":"5319e7"},{"url":"https://api.github.com/repos/facebook/react/labels/Type:%20Feature%20Request","name":"Type: Feature Request","color":"c7def8"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2014-02-20T09:07:13Z","updated_at":"2016-02-10T20:54:18Z","closed_at":"2016-02-10T20:54:18Z","body":"What do you think of a \"mixed in\" callback for mixins which is called at the moment of applying a mixin to a component with component's spec as an argument?\r\n\r\n```\r\nvar SomeTrickeryMixin = {\r\n  mixin: function(spec) {\r\n    // redefine .render() or do some other dangerous things\r\n  },\r\n\r\n  ...\r\n};\r\n```\r\n\r\nBesides overriding `.render()` such callbacks can be used when you want to implement a \"custom policy\" for some methods (wrap method, call-before, call-after, ...).\r\n\r\nAs a downside that would make things a little less straightforward from a (mixin) user perspective."},"comment":{"url":"https://api.github.com/repos/facebook/react/issues/comments/182578655","html_url":"https://github.com/facebook/react/issues/1138#issuecomment-182578655","issue_url":"https://api.github.com/repos/facebook/react/issues/1138","id":182578655,"user":{"login":"jimfb","id":9595985,"avatar_url":"https://avatars.githubusercontent.com/u/9595985?v=3","gravatar_id":"","url":"https://api.github.com/users/jimfb","html_url":"https://github.com/jimfb","followers_url":"https://api.github.com/users/jimfb/followers","following_url":"https://api.github.com/users/jimfb/following{/other_user}","gists_url":"https://api.github.com/users/jimfb/gists{/gist_id}","starred_url":"https://api.github.com/users/jimfb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jimfb/subscriptions","organizations_url":"https://api.github.com/users/jimfb/orgs","repos_url":"https://api.github.com/users/jimfb/repos","events_url":"https://api.github.com/users/jimfb/events{/privacy}","received_events_url":"https://api.github.com/users/jimfb/received_events","type":"User","site_admin":false},"created_at":"2016-02-10T20:54:17Z","updated_at":"2016-02-10T20:54:17Z","body":"As per https://github.com/facebook/react/pull/1305#issuecomment-99318251, mixins are a legacy feature and we're not really adding things to them at this point.  This issue has been untouched for over a year, closing due to inactivity."}}