[Android Bootcamp] App Ready for Review

[azoff]

App's ready, please review. /cc @nesquena

Pain points:

• Importing PullToRefresh (or any external android module) into Android Studio. Solution ended up being two parts:
  • Get the editor to recognize the module
  • Add the module folder to the project
  • Expose the module to the editor by using the "Import Modules" tool in the "Libraries..." context menu.
  • Mark the newly imported Module as a dependency of the Twitter App
  • Get gradle to compile and package the module
  • Ensure that the editor is using the correct version of Gradle (I had to download 1.6)
  • In gradle.settings, make a reference to the module folder (see the file for an example)
  • In build.gradle mark the project reference for compilation
• Adding relationships in ActiveAndroid
  • Turns out you can't set a relationship member (i.e. Tweet.user) with a Model that has not previously been save()d.
  • The result is a super-vague NullPointerException

[nesquena]

Jonathan, :+1: excellent submission. A few notes after checking out the code:

• Nice to see you used PullToRefresh library
• Good that you only have a single method which calls getHomeTimeline
• Good to see you refreshed the timeline after compose
• Nice touch with the character count being calculated for Compose
• Good to see you added ActiveAndroid annotations to the models

Here's a detailed Project 3 Feedback Guide here which covers the most common issues with this submitted project. Read through the feedback guide point-by-point to determine how you could improve your submission.

Let us know if you have any other thoughts or questions about this assignment. Hopefully by now you feel pretty comfortable with all the major pieces to basic Android apps (Views, Controllers, Models, Authentication, API Communication, Preferences, ActionBar, et al) and see how they all fit together. We are close to a turning point in the course where you should be hitting a "critical mass" for your Android knowledge.

[nesquena]

Turns out you can't set a relationship member (i.e. Tweet.user) with a Model that has not previously been save()d.

Thanks for the pain notes, added a note about the saving issue to helpful hints. I would like to fix that issue in ActiveAndroid eventually. IMO it should autosave the model rather than error out.