{"action":"created","issue":{"url":"https://api.github.com/repos/justcool393/TotesMessenger/issues/3","labels_url":"https://api.github.com/repos/justcool393/TotesMessenger/issues/3/labels{/name}","comments_url":"https://api.github.com/repos/justcool393/TotesMessenger/issues/3/comments","events_url":"https://api.github.com/repos/justcool393/TotesMessenger/issues/3/events","html_url":"https://github.com/justcool393/TotesMessenger/pull/3","id":66133933,"number":3,"title":"Total rewrite of TotesMessenger","user":{"login":"hidde-jan","id":171505,"avatar_url":"https://avatars.githubusercontent.com/u/171505?v=3","gravatar_id":"","url":"https://api.github.com/users/hidde-jan","html_url":"https://github.com/hidde-jan","followers_url":"https://api.github.com/users/hidde-jan/followers","following_url":"https://api.github.com/users/hidde-jan/following{/other_user}","gists_url":"https://api.github.com/users/hidde-jan/gists{/gist_id}","starred_url":"https://api.github.com/users/hidde-jan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hidde-jan/subscriptions","organizations_url":"https://api.github.com/users/hidde-jan/orgs","repos_url":"https://api.github.com/users/hidde-jan/repos","events_url":"https://api.github.com/users/hidde-jan/events{/privacy}","received_events_url":"https://api.github.com/users/hidde-jan/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-04-03T12:02:31Z","updated_at":"2015-04-03T20:00:45Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/justcool393/TotesMessenger/pulls/3","html_url":"https://github.com/justcool393/TotesMessenger/pull/3","diff_url":"https://github.com/justcool393/TotesMessenger/pull/3.diff","patch_url":"https://github.com/justcool393/TotesMessenger/pull/3.patch"},"body":"Hi justcool,\r\n\r\nLike we discussed in our private messages on reddit, here is the rewrite I was talking about. I understand that it might be a bit strange to just throw away everything you've made. At first I tried to just do some refactoring, but in the end I couldn't really figure out how everything fit together.\r\n\r\nOf course, I understand if you don't want to use the code. That's not a problem. I don't think you need to worry about hosting it on heroku. The sqlite database is recreated from scratch every time it gets lost.\r\n\r\nCheers,\r\nHJ\r\n\r\n----\r\n\r\nThis rewrite changes the architecture of the bot to a more class\r\nbased one. Some of the changes:\r\n\r\n- Split logic into different classes.\r\n- Use a sqlite db, which can be recreated if it's missing.\r\n- Try to minimize global state (I know, I know...)\r\n- Link and Source now have a very similar api.\r\n- Reduce the amount of api requests that need to be made.\r\n- etc."},"comment":{"url":"https://api.github.com/repos/justcool393/TotesMessenger/issues/comments/89406955","html_url":"https://github.com/justcool393/TotesMessenger/pull/3#issuecomment-89406955","issue_url":"https://api.github.com/repos/justcool393/TotesMessenger/issues/3","id":89406955,"user":{"login":"hidde-jan","id":171505,"avatar_url":"https://avatars.githubusercontent.com/u/171505?v=3","gravatar_id":"","url":"https://api.github.com/users/hidde-jan","html_url":"https://github.com/hidde-jan","followers_url":"https://api.github.com/users/hidde-jan/followers","following_url":"https://api.github.com/users/hidde-jan/following{/other_user}","gists_url":"https://api.github.com/users/hidde-jan/gists{/gist_id}","starred_url":"https://api.github.com/users/hidde-jan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hidde-jan/subscriptions","organizations_url":"https://api.github.com/users/hidde-jan/orgs","repos_url":"https://api.github.com/users/hidde-jan/repos","events_url":"https://api.github.com/users/hidde-jan/events{/privacy}","received_events_url":"https://api.github.com/users/hidde-jan/received_events","type":"User","site_admin":false},"created_at":"2015-04-03T20:00:45Z","updated_at":"2015-04-03T20:00:45Z","body":"This rewrite also fixes some of the other open issues for this project. "}}