[Intro to iOS] Week3 Project Twitter - Review my app

[JackalWang]

My app is complete. please review. /cc @codepath

[timothy1ee]

:+1: nice work. With this assignment, we've now explored all the main patterns for building MVC clients! If this were 2009, you would be well on your way to building most apps that you could find in the app store. Over the next few weeks, we'll be focusing on custom views and view controllers to implement the interactions and visual effects that we find in more modern iOS apps.

Here's a checklist of things that I'm looking for in this project. You should review the checklist, and make any changes to your project, if necessary.

• In Objective-C, review your .h files. In Swift, review the properties and methods that you've made public. Your classes should reveal a minimum interface and only expose things that must be used by other classes.
• Each .xib file or Storyboard scene should have Auto Layout constraints specified for each view.
• There should be no yellow Auto Layout warnings, even if the layout seems to work. Auto Layout warnings are like having bad HTML. The browser will try to render it, and it might even look right, but it won't be fully predictable.
• In your custom TweetCell, there is a conditionally present label that starts with "retweeted by ...". Did you create an outlet for that constraint and programatically modify the constant depending if the label is present or hidden?
• Did you handle reply/retweet/favorite actions from within the TweetCell? This is an example where it's useful for your model to be able to handle network actions, so you can simply wire up the buttons to a method like [tweet retweet] (Objective-C) or tweet.retweet() (Swift). You can even pass in an optional closure if you want to handle a failure of the network event.
• If you're using Objective-C, you can consider using Mantle, a lightweight ORM that eliminates a lot of boilerplate. It's also compatible with Core Data if your project requires that.
• If you're using Swift, you can consider using SwiftyJSON for easier deserialization of the JSON response.
• In your ComposeViewController, did you pass back the Tweet after it was successfully created? This will allow you to locally insert newly created tweets so you don't have to do a network refresh to see the new tweets.