codepathreview
commented
10 years ago :+1: nice work Qingdi. A few notes after checking out the code:
- Good to see you were able to format the timestamp cleanly
- Good to see you refreshed the timeline after composing a new tweet
- Glad to see you made the compose activity into a dialog instead
- Consider adding progress bars to your app whenever a network request goes out
- Consider adding pull to refresh for the home timeline
- Consider adding local persistence of tweets by using ActiveAndroid in the Tweet and User.
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, ActionBar, Navigation, Models, Authentication, API Communication, Persistence, et al) and see how they all fit together. We are close now to a turning point in the course where you should be hitting a "critical mass" towards your knowledge of Android.
/cc @nesquena @thecodepath Completed the project with the following features: Required: User can sign in to Twitter using OAuth login Required: User can view the tweets from their home timeline Required: User can compose a new tweet Advanced: Improve the user interface and theme the app to feel "twitter branded" Compose activity is replaced with a modal overlay