jonidacali / TwitterClient

0 stars 0 forks source link

Twitter Client (with fragments) - Review request #2

Open jonidacali opened 10 years ago

jonidacali commented 10 years ago

Thanks in advance for your feedback! @nesquena @thecodepath

PS. Please see twitter_v2.mp4 for this week's video

jonidacali commented 10 years ago

Please provide some input on the way I am displaying the newly created tweet in the home timeline. I realized quiet late that it was not working and the approach I took seems highly "hack-y" to me.

Initially I tried Tweet.addAll(arrayObject). The problem with that was that the array included the latest tweet but when inspecting it it would show as a tweet@null (even thought I would see all the elements of the tweet model initialized). So I ended up following the other approach... What would be the right way to pass the new tweet in the adapter?

codepathreview commented 10 years ago

:+1: nice work Jona. A few notes after checking out the code:

Here's a detailed Project 4 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 might be able to improve your submission.

This week (Week 5), we are going to cover the last major piece to the Android puzzle and that is using the hardware and SDK components such as the camera, photo gallery, location, maps, etc. After that, Week 6 and week 7 we will be covering a few important intermediate topics such as more about styling and animation as well as testing.

Following the bootcamp, we are going to have a demo day to celebrate the progress you've all made with our next batch of Android students and multiple companies attending to see the group projects that you all have built. We are going to help however we can over the next few weeks to get the team project apps in shape for that.