{"action":"created","issue":{"url":"https://api.github.com/repos/baristaze/Hamburger/issues/1","labels_url":"https://api.github.com/repos/baristaze/Hamburger/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/baristaze/Hamburger/issues/1/comments","events_url":"https://api.github.com/repos/baristaze/Hamburger/issues/1/events","html_url":"https://github.com/baristaze/Hamburger/issues/1","id":83334914,"number":1,"title":"[iOS Bootcamp] Assignment 4 - Hamburger","user":{"login":"baristaze","id":2823076,"avatar_url":"https://avatars.githubusercontent.com/u/2823076?v=3","gravatar_id":"","url":"https://api.github.com/users/baristaze","html_url":"https://github.com/baristaze","followers_url":"https://api.github.com/users/baristaze/followers","following_url":"https://api.github.com/users/baristaze/following{/other_user}","gists_url":"https://api.github.com/users/baristaze/gists{/gist_id}","starred_url":"https://api.github.com/users/baristaze/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/baristaze/subscriptions","organizations_url":"https://api.github.com/users/baristaze/orgs","repos_url":"https://api.github.com/users/baristaze/repos","events_url":"https://api.github.com/users/baristaze/events{/privacy}","received_events_url":"https://api.github.com/users/baristaze/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-06-01T06:04:38Z","updated_at":"2015-06-06T02:33:06Z","closed_at":null,"body":"My app is complete, please review. //cc @codepathreview"},"comment":{"url":"https://api.github.com/repos/baristaze/Hamburger/issues/comments/109504547","html_url":"https://github.com/baristaze/Hamburger/issues/1#issuecomment-109504547","issue_url":"https://api.github.com/repos/baristaze/Hamburger/issues/1","id":109504547,"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-06-06T02:33:06Z","updated_at":"2015-06-06T02:33:06Z","body":"Nice work! We added this homework so that you'll have the experience building an app that's more similar in complexity to a production app. It's also useful to have practical experience building a container view controller, which is a more advanced topic. The optional account switching feature is interesting because you have to think about how TwitterClient is caching the access token to make sure it works correctly for multiple accounts. Displaying the account switching screen is implemented using custom view controller transitions.\r\n\r\nA few notes after checking out the code:\r\n\r\n- Nice work encapsulating the properties\r\n- Note that you should resolve all the Auto Layout constraint\r\n- Consider refactoring the profile view controller so that it shares code with the TimelineViewController. There are two general strategies: you can subclass the TimelineViewController or create a separate class called a TimelineController (not a view controller). TimelineController is a dataSource and a delegate to a UITableView and knows how to show a feed of tweets.\r\n- Nice work encapsulating the properties of the custom cells that you added.\r\n- You should consider what a more generic design for your hamburger controller might look like. Right now, it's tightly bound to this Twitter application.\r\n- You should consider implementing the blurred table view header. There are several approaches here that are all valid. You can implement the header has a table header, a section header, or a cell. My preferred approach is to implement it as a table header, it's the easiest way to flow the image above the content. These detail touches are where you really learn the nooks and crannies of the framework.\r\n\r\nCongratulations on finishing the final assignment! It's impressive to realize that in just four weeks, you've gone from building a simple tip calculator app to a polished MVC client."}}