Yelp App submission!

[yatchoi]

My app is complete! Please review :)

@codepathreview

[nickai]

:+1: Excellent work Yat! It's great to hear you enjoyed this assignment and it really shows with all the extensions you added!

The primary goals of this homework are to learn how to use AutoLayout to build adaptive UI's that work across many different screen sizes and orientations and to get practice using the delegate pattern. If you haven't already, I recommend going back and running your Yelp app again while rotating the device / simulator to ensure your AutoLayout constraints function as expected.

A couple notes after checking out your code / gif:

• I noticed that your restaurant names aren't wrapping (they are being ellipsized) when the name is really long. You'll want to set up your AutoLayout constraints such that the restaurant names can take multiple lines and the overall cell height increases.
• Great to see you tackle maps and the detail screen!
• We haven't covered working with location (and won't get to it in the fast track version of the class), but you'll typically see most apps use a singleton MyLocationManager that handles all the location needs for the app. Then the individual view controllers can interact with that singleton instance to get the latest location, etc. This singleton is created from the AppDelegate so it's tied to the app lifecycle instead of the view controller lifecycle.

Here's our general Yelp assignment feedback guide:

• Did you make sure to use AutoLayout for all your layouts? All screens should contain AutoLayout constraints. Make sure to resolve any AutoLayout constraint warnings.
• Did you try rotating the device / simulator to make sure your AutoLayout constraints are set up properly? To rotate the simulator you can use Cmd-Left Arrow or Cmd-Right Arrow. Rotating the screen is a great way to see if you set up your constraints properly.
• Did you make sure to put all image assets inside Images.xcassets? Using the asset catalog is the best practice for storing images. This also lets you keep multiple versions of each image for the necessary screen resolutions (1x, 2x, and 3x).
• Did you configure your custom Business cell to have a single YelpBusiness property? Your custom Business cell should have a property of type YelpBusiness. In the custom setter, you should configure the various labels and images. This decouples your custom cells from the table view controllers.
• Did you represent the Yelp Filters using custom Model classes? Building custom model classes to represent entities in your app makes them easier to work with and reuse throughout your app.
• Did you create a FiltersViewControllerDelegate? The main view controller and the filters view controller should be connected via a custom filter delegate. This is a popular pattern for handling events between view controllers.
• Did you properly create a form in the filter view controller? Make sure the cells are being recycled and dequeued and appropriately linked to the model. See this popular form library for an alternative to implementing forms manually.
• Did you handle the case where a user cancels out of the Yelp Filters screen? You'll need to make sure you are editing a copy of the Yelp filters when the user edits settings on the Filters page so that when the user cancels out of this screen you can restore the original settings. While connecting the filter view controller directly to a global filter settings is convenient, the downside is if you cancel the filters, the values have already been saved.
• Did you animate the dropdown of expandable filter settings? TableViews have a set of methods that allow you to reload/insert/delete rows and sections of a TableView with an animation (one example is reloadRowsAtIndexPaths:withRowAnimation:).
• Did you configure your table in FiltersViewController using multiple sections? By having multiple sections in your TableView, you can use tableView:titleForHeaderInSection: to add a title to each section.