{"action":"created","issue":{"url":"https://api.github.com/repos/NatalieZhang/AndBootcampIns/issues/3","labels_url":"https://api.github.com/repos/NatalieZhang/AndBootcampIns/issues/3/labels{/name}","comments_url":"https://api.github.com/repos/NatalieZhang/AndBootcampIns/issues/3/comments","events_url":"https://api.github.com/repos/NatalieZhang/AndBootcampIns/issues/3/events","html_url":"https://github.com/NatalieZhang/AndBootcampIns/issues/3","id":114528694,"number":3,"title":"[Android-bootcamp] Assignment 3-Instagram app search by users/tags - ready for review","user":{"login":"NatalieZhang","id":7528896,"avatar_url":"https://avatars.githubusercontent.com/u/7528896?v=3","gravatar_id":"","url":"https://api.github.com/users/NatalieZhang","html_url":"https://github.com/NatalieZhang","followers_url":"https://api.github.com/users/NatalieZhang/followers","following_url":"https://api.github.com/users/NatalieZhang/following{/other_user}","gists_url":"https://api.github.com/users/NatalieZhang/gists{/gist_id}","starred_url":"https://api.github.com/users/NatalieZhang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/NatalieZhang/subscriptions","organizations_url":"https://api.github.com/users/NatalieZhang/orgs","repos_url":"https://api.github.com/users/NatalieZhang/repos","events_url":"https://api.github.com/users/NatalieZhang/events{/privacy}","received_events_url":"https://api.github.com/users/NatalieZhang/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-11-02T05:42:21Z","updated_at":"2015-11-02T08:02:40Z","closed_at":null,"body":"Please review. @codepathreview @codepath"},"comment":{"url":"https://api.github.com/repos/NatalieZhang/AndBootcampIns/issues/comments/152946486","html_url":"https://github.com/NatalieZhang/AndBootcampIns/issues/3#issuecomment-152946486","issue_url":"https://api.github.com/repos/NatalieZhang/AndBootcampIns/issues/3","id":152946486,"user":{"login":"codepathreview","id":7917093,"avatar_url":"https://avatars.githubusercontent.com/u/7917093?v=3","gravatar_id":"","url":"https://api.github.com/users/codepathreview","html_url":"https://github.com/codepathreview","followers_url":"https://api.github.com/users/codepathreview/followers","following_url":"https://api.github.com/users/codepathreview/following{/other_user}","gists_url":"https://api.github.com/users/codepathreview/gists{/gist_id}","starred_url":"https://api.github.com/users/codepathreview/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/codepathreview/subscriptions","organizations_url":"https://api.github.com/users/codepathreview/orgs","repos_url":"https://api.github.com/users/codepathreview/repos","events_url":"https://api.github.com/users/codepathreview/events{/privacy}","received_events_url":"https://api.github.com/users/codepathreview/received_events","type":"User","site_admin":false},"created_at":"2015-11-02T08:02:40Z","updated_at":"2015-11-02T08:02:40Z","body":"Good job finishing all the stories. Some feedback after looking at the code -\r\n\r\n* Good job putting a global helper method to retrieve the InstagramClient in Application class\r\n* It's better to do any assignment of subviews to fields in `onViewCreated`. This is because the framework does an automatic null check for you to ensure that your Fragment's view hierarchy has been created and inflated (if using an XML layout file) properly.\r\n* Good job updating the adapter by calling notifyDataSetChanged instead of creating a new adapter after fetching completes\r\n* It's always a good idea to display an indeterminate or determinate progress bar to the user when potentially long running tasks are happening in the background, especially those that update views. See **[this guide on implementing a ProgressBarView](https://guides.codepath.com/android/Handling-ProgressBars)**.\r\n* It is recommended to put integer dimension, sp, dp etc to 'dimens.xml'"}}