{"action":"created","issue":{"url":"https://api.github.com/repos/meanjs/mean/issues/1687","repository_url":"https://api.github.com/repos/meanjs/mean","labels_url":"https://api.github.com/repos/meanjs/mean/issues/1687/labels{/name}","comments_url":"https://api.github.com/repos/meanjs/mean/issues/1687/comments","events_url":"https://api.github.com/repos/meanjs/mean/issues/1687/events","html_url":"https://github.com/meanjs/mean/pull/1687","id":197422235,"number":1687,"title":"[v2] MEAN.JS API - Introducing the SequelizeJS library","user":{"login":"lirantal","id":316371,"avatar_url":"https://avatars.githubusercontent.com/u/316371?v=3","gravatar_id":"","url":"https://api.github.com/users/lirantal","html_url":"https://github.com/lirantal","followers_url":"https://api.github.com/users/lirantal/followers","following_url":"https://api.github.com/users/lirantal/following{/other_user}","gists_url":"https://api.github.com/users/lirantal/gists{/gist_id}","starred_url":"https://api.github.com/users/lirantal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lirantal/subscriptions","organizations_url":"https://api.github.com/users/lirantal/orgs","repos_url":"https://api.github.com/users/lirantal/repos","events_url":"https://api.github.com/users/lirantal/events{/privacy}","received_events_url":"https://api.github.com/users/lirantal/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"lirantal","id":316371,"avatar_url":"https://avatars.githubusercontent.com/u/316371?v=3","gravatar_id":"","url":"https://api.github.com/users/lirantal","html_url":"https://github.com/lirantal","followers_url":"https://api.github.com/users/lirantal/followers","following_url":"https://api.github.com/users/lirantal/following{/other_user}","gists_url":"https://api.github.com/users/lirantal/gists{/gist_id}","starred_url":"https://api.github.com/users/lirantal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lirantal/subscriptions","organizations_url":"https://api.github.com/users/lirantal/orgs","repos_url":"https://api.github.com/users/lirantal/repos","events_url":"https://api.github.com/users/lirantal/events{/privacy}","received_events_url":"https://api.github.com/users/lirantal/received_events","type":"User","site_admin":false},"assignees":[{"login":"lirantal","id":316371,"avatar_url":"https://avatars.githubusercontent.com/u/316371?v=3","gravatar_id":"","url":"https://api.github.com/users/lirantal","html_url":"https://github.com/lirantal","followers_url":"https://api.github.com/users/lirantal/followers","following_url":"https://api.github.com/users/lirantal/following{/other_user}","gists_url":"https://api.github.com/users/lirantal/gists{/gist_id}","starred_url":"https://api.github.com/users/lirantal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lirantal/subscriptions","organizations_url":"https://api.github.com/users/lirantal/orgs","repos_url":"https://api.github.com/users/lirantal/repos","events_url":"https://api.github.com/users/lirantal/events{/privacy}","received_events_url":"https://api.github.com/users/lirantal/received_events","type":"User","site_admin":false}],"milestone":null,"comments":0,"created_at":"2016-12-23T20:20:10Z","updated_at":"2016-12-24T00:59:24Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/meanjs/mean/pulls/1687","html_url":"https://github.com/meanjs/mean/pull/1687","diff_url":"https://github.com/meanjs/mean/pull/1687.diff","patch_url":"https://github.com/meanjs/mean/pull/1687.patch"},"body":"* Introducing the SequelizeJS ORM for traditional SQL databases support\r\n* Adding the MySQL library binding for SequelizeJS\r\n* Removing of `node-inspector` library which is not compatible with \r\n* Refactoring of Mongoose,  `app.js`, models structure, and config files \r\n\r\nThe new server instantiation is based on async/await and requires Node v7 (Babel isn't yet introduced in the codebase), so run with:\r\n\r\n```\r\nnode --harmony-async-await server.js\r\n```\r\n\r\nCode Review: https://github.com/meanjs/mean/commit/7a5ffa3ae456abdf2c2f5eeba236453e4a867e2f"},"comment":{"url":"https://api.github.com/repos/meanjs/mean/issues/comments/269061358","html_url":"https://github.com/meanjs/mean/pull/1687#issuecomment-269061358","issue_url":"https://api.github.com/repos/meanjs/mean/issues/1687","id":269061358,"user":{"login":"mleanos","id":5613734,"avatar_url":"https://avatars.githubusercontent.com/u/5613734?v=3","gravatar_id":"","url":"https://api.github.com/users/mleanos","html_url":"https://github.com/mleanos","followers_url":"https://api.github.com/users/mleanos/followers","following_url":"https://api.github.com/users/mleanos/following{/other_user}","gists_url":"https://api.github.com/users/mleanos/gists{/gist_id}","starred_url":"https://api.github.com/users/mleanos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mleanos/subscriptions","organizations_url":"https://api.github.com/users/mleanos/orgs","repos_url":"https://api.github.com/users/mleanos/repos","events_url":"https://api.github.com/users/mleanos/events{/privacy}","received_events_url":"https://api.github.com/users/mleanos/received_events","type":"User","site_admin":false},"created_at":"2016-12-24T00:59:24Z","updated_at":"2016-12-24T00:59:24Z","body":"I think we should keep an open dialog on the changes and/or plans going into this new branch. I'm interested to know your thoughts on how we should handle the business logic that will utilize these models inside our API methods. \r\n\r\nI've done some work recently with building a Node.js API from scratch using Typescript. I've managed to separate my models from the database logic, but the service layer that I built relies heavily on the DB I'm using (currently MongoDB), since the way we interact with the database is very specific to what data storage we use.\r\n"}}
{"actor":{"display_login":"mleanos"}}