{"action":"closed","issue":{"url":"https://api.github.com/repos/JanuszHain/SocialMediaWatcher/issues/1","repository_url":"https://api.github.com/repos/JanuszHain/SocialMediaWatcher","labels_url":"https://api.github.com/repos/JanuszHain/SocialMediaWatcher/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/JanuszHain/SocialMediaWatcher/issues/1/comments","events_url":"https://api.github.com/repos/JanuszHain/SocialMediaWatcher/issues/1/events","html_url":"https://github.com/JanuszHain/SocialMediaWatcher/issues/1","id":175119919,"number":1,"title":"Code Review","user":{"login":"MBratkowski","id":6844181,"avatar_url":"https://avatars.githubusercontent.com/u/6844181?v=3","gravatar_id":"","url":"https://api.github.com/users/MBratkowski","html_url":"https://github.com/MBratkowski","followers_url":"https://api.github.com/users/MBratkowski/followers","following_url":"https://api.github.com/users/MBratkowski/following{/other_user}","gists_url":"https://api.github.com/users/MBratkowski/gists{/gist_id}","starred_url":"https://api.github.com/users/MBratkowski/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MBratkowski/subscriptions","organizations_url":"https://api.github.com/users/MBratkowski/orgs","repos_url":"https://api.github.com/users/MBratkowski/repos","events_url":"https://api.github.com/users/MBratkowski/events{/privacy}","received_events_url":"https://api.github.com/users/MBratkowski/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-09-05T19:31:11Z","updated_at":"2016-09-06T16:16:29Z","closed_at":"2016-09-06T16:16:29Z","body":"- You don't have any application structure. You could separate different type classes. Example if you have a adapteres class you create a new package with name 'adapter, if you have a model class put is in 'model' package. When you separate your classes to package your application structure are better, testable, maintainability.\r\n- If you use a Rx you do not use a EventBus library.\r\n- Your project do not use a Object Programming Paradidgmats, like abstraction or composition. \r\n- Use a Threads to manage task is a bad idea when you have a Rx library.\r\n- Your application are not separate to corners, you don't have a application layers like - model, view or presenter. Read more in topic `Android MVP design pattern`."}}
{"actor":{"display_login":"JanuszHain"}}