{"issue":{"created_at":"2011-11-02T13:29:16Z","state":"open","updated_at":"2011-11-18T10:23:36Z","user":{"gravatar_id":"e01e261ed76aa771af08ad836726e13a","id":756520,"url":"https://api.github.com/users/kmalakoff","login":"kmalakoff","avatar_url":"https://secure.gravatar.com/avatar/e01e261ed76aa771af08ad836726e13a?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"comments":5,"body":"Hi Paul,\r\n\r\nI've done a clean repository with the minimum changes needed for Backbone-Articulation. \r\n\r\nMuch cleaner and easier to understand.\r\n\r\nWould you consider merging this into the main branch? It is non-destructive.\r\n\r\nCheers,\r\n\r\nKevin","assignee":null,"id":2121337,"number":45,"labels":[],"pull_request":{"patch_url":"https://github.com/PaulUithol/Backbone-relational/pull/45.patch","html_url":"https://github.com/PaulUithol/Backbone-relational/pull/45","diff_url":"https://github.com/PaulUithol/Backbone-relational/pull/45.diff"},"title":"Minimal changes for Backbone-Relational","closed_at":null,"html_url":"https://github.com/PaulUithol/Backbone-relational/issues/45","milestone":null,"url":"https://api.github.com/repos/PaulUithol/Backbone-relational/issues/45"},"action":"created","comment":{"created_at":"2011-11-18T10:23:36Z","updated_at":"2011-11-18T10:23:36Z","user":{"gravatar_id":"b5c0cf186026bae6da5144f24573269f","id":670670,"url":"https://api.github.com/users/PaulUithol","login":"PaulUithol","avatar_url":"https://secure.gravatar.com/avatar/b5c0cf186026bae6da5144f24573269f?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"body":"Haven't found the time yet to look into this issue properly, but good to hear you're still improving!\r\n\r\nFor the time being, monkey-patching it in your lib might just be the best solution. Actually, I rather like that method, since JavaScript is quite well suited to that style of aspect-oriented programming. I'm doing the same for most Backbone methods, since I need to override a lot of behavior as well (and I do agree with them that that code doesn't belong in Backbone itself).\r\n\r\nThe only issue is that the loading order does become quite important; Backbone-Relational should be loaded before your plugin, or your modification will be messed up.","id":2787674,"url":"https://api.github.com/repos/PaulUithol/Backbone-relational/issues/comments/2787674"}}