{"action":"created","issue":{"url":"https://api.github.com/repos/android10/Android-CleanArchitecture/issues/44","labels_url":"https://api.github.com/repos/android10/Android-CleanArchitecture/issues/44/labels{/name}","comments_url":"https://api.github.com/repos/android10/Android-CleanArchitecture/issues/44/comments","events_url":"https://api.github.com/repos/android10/Android-CleanArchitecture/issues/44/events","html_url":"https://github.com/android10/Android-CleanArchitecture/issues/44","id":105777715,"number":44,"title":"UserListPresenter#onUserClicked naming","user":{"login":"mateuyabar","id":1798264,"avatar_url":"https://avatars.githubusercontent.com/u/1798264?v=3","gravatar_id":"","url":"https://api.github.com/users/mateuyabar","html_url":"https://github.com/mateuyabar","followers_url":"https://api.github.com/users/mateuyabar/followers","following_url":"https://api.github.com/users/mateuyabar/following{/other_user}","gists_url":"https://api.github.com/users/mateuyabar/gists{/gist_id}","starred_url":"https://api.github.com/users/mateuyabar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mateuyabar/subscriptions","organizations_url":"https://api.github.com/users/mateuyabar/orgs","repos_url":"https://api.github.com/users/mateuyabar/repos","events_url":"https://api.github.com/users/mateuyabar/events{/privacy}","received_events_url":"https://api.github.com/users/mateuyabar/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-09-10T10:16:19Z","updated_at":"2015-09-11T15:12:42Z","closed_at":null,"body":"Hi, very interesting post.\r\n\r\nI have a method naming question/issue.\r\n\r\nIn the UserListPresenter, you have defined a method *onUserClicked(UserModel userModel)*. I'm not sure if this is a correct name. The idea is that the View can change but the presenter remains the same, but the concept of *UserClicked* is something more of the view. Maybe the view changes and the user is displayed in a double click.\r\n\r\nIn my opinion this method should be called something like *viewUserDetail(UserModel userModel)*.\r\n\r\nThanks,"},"comment":{"url":"https://api.github.com/repos/android10/Android-CleanArchitecture/issues/comments/139571671","html_url":"https://github.com/android10/Android-CleanArchitecture/issues/44#issuecomment-139571671","issue_url":"https://api.github.com/repos/android10/Android-CleanArchitecture/issues/44","id":139571671,"user":{"login":"android10","id":1360604,"avatar_url":"https://avatars.githubusercontent.com/u/1360604?v=3","gravatar_id":"","url":"https://api.github.com/users/android10","html_url":"https://github.com/android10","followers_url":"https://api.github.com/users/android10/followers","following_url":"https://api.github.com/users/android10/following{/other_user}","gists_url":"https://api.github.com/users/android10/gists{/gist_id}","starred_url":"https://api.github.com/users/android10/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/android10/subscriptions","organizations_url":"https://api.github.com/users/android10/orgs","repos_url":"https://api.github.com/users/android10/repos","events_url":"https://api.github.com/users/android10/events{/privacy}","received_events_url":"https://api.github.com/users/android10/received_events","type":"User","site_admin":false},"created_at":"2015-09-11T15:12:42Z","updated_at":"2015-09-11T15:12:42Z","body":"@caseykulm good explanation and totally agree. Thanks for clarifying it. "}}