Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

App is completed. #2

Open
aftabnaqvi opened this issue Oct 9, 2014 · 1 comment
Open

App is completed. #2

aftabnaqvi opened this issue Oct 9, 2014 · 1 comment

Comments

@aftabnaqvi
Copy link
Owner

My app is completed, please review it.

Thanks,
Syed

@nesquena
@thecodepath

@codepathreview
Copy link

👍 Excellent work Syed. A few notes after checking out the code:

  • Nice job with including progress indicators..
  • Good use of view holder pattern inside adapter
  • Good to see you properly communicated the user to the user timeline fragment from the profile activity using newInstance.
  • Nicely done architecting the fragments (base class and extension classes) such that you put as much code and logic as possible in the base class and then used abstract methods to define only the specific behavior where they differ!
  • Excellent clean UI and attention to detail is really impressive
  • Great job getting most of the extras completed
  • Nice use of tag to setup the ability to click the profile image and view a user's profile

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants