jnermano / NYTimesArticlesSearch

Build a New York Times News Search article app which allows a user to find old articles.
0 stars 0 forks source link

Project Feedback! #1

Open codepathreview opened 6 years ago

codepathreview commented 6 years ago

:+1: nice work. Some notes after checking out the code:

Good to see you got sharing working using the ShareActionProvider system. Keep in mind that this is simply leveraging the implicit intent system behind the scenes. Refer to this guide on implicit intents to see other common usages of them.

Let us know if you have any other thoughts or questions about this assignment. The next assignment (Twitter Client) will be especially important since it introduces the majority of the remaining pieces necessary to build a fully functional app including persistence with SQLite and modular view re-use with Fragments.

jnermano commented 6 years ago

Thank you for the review.