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

Finished assignment #73

Closed
wants to merge 2 commits into from
Closed

Finished assignment #73

wants to merge 2 commits into from

Conversation

mullfuchs
Copy link

Deliverable Submission

Please describe your comfort and completeness levels before submitting.

Comfort Level (1-5): 3

Completeness Level (1-5): 4.5 (couldn't get query string in the results url)

What did you think of this deliverable?:
Felt like the starting point sent me in a weird direction. Very challenging trying to figure out how to get the imdbID to link to a page, but after that was solved the rest of it was fairly normal.

Also: appears that Rotten Tomatoes API is locked and you must request an API key, as it seems to no longer be public facing.

@brandiw brandiw closed this Oct 5, 2016
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

Successfully merging this pull request may close these issues.

2 participants