{"action":"opened","issue":{"url":"https://api.github.com/repos/HouseTrip/guidelines/issues/25","labels_url":"https://api.github.com/repos/HouseTrip/guidelines/issues/25/labels{/name}","comments_url":"https://api.github.com/repos/HouseTrip/guidelines/issues/25/comments","events_url":"https://api.github.com/repos/HouseTrip/guidelines/issues/25/events","html_url":"https://github.com/HouseTrip/guidelines/issues/25","id":54785516,"number":25,"title":"Why is this bad?","user":{"login":"cored","id":11719,"avatar_url":"https://avatars.githubusercontent.com/u/11719?v=3","gravatar_id":"","url":"https://api.github.com/users/cored","html_url":"https://github.com/cored","followers_url":"https://api.github.com/users/cored/followers","following_url":"https://api.github.com/users/cored/following{/other_user}","gists_url":"https://api.github.com/users/cored/gists{/gist_id}","starred_url":"https://api.github.com/users/cored/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cored/subscriptions","organizations_url":"https://api.github.com/users/cored/orgs","repos_url":"https://api.github.com/users/cored/repos","events_url":"https://api.github.com/users/cored/events{/privacy}","received_events_url":"https://api.github.com/users/cored/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-01-19T16:39:33Z","updated_at":"2015-01-19T16:39:33Z","closed_at":null,"body":"https://github.com/HouseTrip/guidelines/blob/master/sustainable-testing.md#keep-models-thin-and-never-implement-business-logic-in-models\r\n\r\nI'm wondering why don't you let the presentation logic inside the model for small things like this? Even if the SRP is broken because the model is doing two things persistence and presentation logic why sometimes the balance should point to removing the logic from it? Isn't this will make the code base hard to read and understand if you have a lot of smaller pieces to jump into?"}}