nidhi1608 / TwitterClient

Twitter client using twitter REST API.
4 stars 2 forks source link

App review #1

Open nidhi1608 opened 10 years ago

nidhi1608 commented 10 years ago

/cc @nesquena

The app is feature complete, please review. Features, screenshots and time spent can be found in the attached README file.

Note: For the compose a tweet view, it wasn't clear if the original twitter app uses an activity or full screen modal dialog. So, I just took the simple route here and chose to work with an activity.

Known issues:

  1. Favorite button functionality is not implemented yet, so clicking on the star button doesn't do anything in the list view as well as detail view.

Thanks, Nidhi

nesquena commented 10 years ago

Want to make a special note here since you submitted an excellent project, implementing many of the optionals. I fully understand and appreciate that most everyone in this class has full-time work and many other obligations in their lives, which makes those who are able to submit polished versions by the deadline earn my respect that much more. Really hope you found this project to be a great learning experience.

A few notes after checking out the code:

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.