[Yahoo Android] Review Project 3 - Twitter App Client

[skazik]

/cc @nesquena @timothy1ee

This submission is from Sergei Kazantsev. I completed the following user tasks:

User can sign in using OAuth login flow - done User can view last 25 tweets from their home timeline - done User can compose a new tweet -done 80% the following improvements are needed:

• automatic keyboard appearance (in fact it appears on real mobile device and if emulator is launched without hardware keyboard)
• multiline text input (TODO)
• image, text, info - missing

User can click a “Compose” icon in the Action Bar on the top right - done User will have a Compose view opened - done User can enter a message and hit a button to Post - done User should be taken back to home timeline with new tweet visible -done partially:

• it goes back and refresh the timeline
• tweet is not posted due to read only access of the application (given by default; have no comments) so the post fails with an error": {"request":"\/1.1\/statuses\/update.json","error":"Read-only application cannot POST."}

Additional UI/Navigation/Graphics recommendations are not implemented; last week my team started an important Yahoo! integration project with 3rd party vendor required full team attention, working "long hours" and even during whole last weekend. I will return to the recommended app additions as soon as I have time.

[nesquena]

tweet is not posted due to read only access of the application (given by default; have no comments) so the post fails with an error": {"request":"\/1.1\/statuses\/update.json","error":"Read-only application cannot POST."}

See my reply on canvas and update your app to have Read and Write permissions on the twitter dev applications page.

[nesquena]

:+1: Nice work overall. A few thoughts:

• Consider adding endless pagination to the timeline
• Good to see you refreshed the timeline after compose
• Consider adding the character count for Compose Activity to limit to 140. See this event handler for more details.
• Consider removing BaseModel from Tweet and User and follow this json to model approach instead which doesn't require BaseModel

I have provided 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.