-
Notifications
You must be signed in to change notification settings - Fork 13
Sven Otto - final project plan #1
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
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,56 @@ | ||
| Project Name: | ||
| "NYT Reading List" | ||
|
|
||
| Elevator Pitch: | ||
| The who: | ||
| Users hoping to create a reading list with various resources available through the NYT. | ||
|
|
||
| The value: | ||
| - Being able to conveniently search a large number of channels within the NYT media universe. | ||
| - Being able to create a personal reading list without having to create a NYT account. | ||
|
|
||
|
|
||
| Dependencies: | ||
| - NYT API (https://developer.nytimes.com/apis) + fetch API | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Note that the API key for the NYT isn't supposed to be exposed in a React app. It's recommended to call that API from a web service that doesn't expose the key. But if you don't have time to do that, it's fine. Just keep in mind it could be deactivated if someone uses/abuses the API with the key. |
||
| - npm modules | ||
| - PropTypes | ||
| - material UI | ||
| - font awesome icons | ||
| - react router | ||
| - Jest & Enzyme (if bonus work and more elaborate testing) | ||
| - Firebase or local storage (if doing some of the bonus work) | ||
|
|
||
| Tasks: | ||
| - Setup create-react-app scaffold | ||
| - Build basic UI with homepage and navbar routing to different search components | ||
| - Fetch NYT api search results | ||
| - Display search results on page including basic preview information | ||
| - Implement button on each result to add said result to reading list component | ||
| - Build reading list component to add/remove saved items and display them | ||
|
|
||
| Bonus: | ||
| - Allow display/hiding of reading list | ||
| - If items are already on list, provide feedback on page | ||
| - Implement local storage/ Firebase for reading list | ||
| - Offer pagination to go through search results X at a time | ||
| - Add "more by this author" search function | ||
| - Offer more comprehensive error handling | ||
| - Write more extensive tests | ||
|
|
||
| Timeline: | ||
|
|
||
| by May 24: | ||
| - setup react-app scaffold | ||
| - basic UI | ||
|
|
||
| by May 31: | ||
| - fetch commands | ||
| - display results | ||
|
|
||
| by June 7: | ||
| - reading list component | ||
| - storage solution | ||
|
|
||
| by June 14: | ||
| - bonus items | ||
| - deploy and submit final project | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍