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

Api user stories #15

Merged
merged 30 commits into from
Oct 11, 2023
Merged

Api user stories #15

merged 30 commits into from
Oct 11, 2023

Conversation

elizakeating
Copy link
Owner

Description

Implement the Discover Movies Page, Movie Results Page, and Movie Details page user stories.

Discover Movies Page:

  • added button to see the top rated movies
  • added a text field to enter a keyword to search with and a button to submit that

Movie Results Page:

  • implemented an api call to either the top rated endpoint or the search endpoint to display max 20 movies
  • movies displayed have a title and a vote average shown
  • movie name links to that movies details page

Movie Details Page:

  • added a button to return to the discover page
  • added a button to create a new viewing part for that user and specific movie
  • implemented three api calls to gather details about the movie, cast, and reviews

Also implemented encryption for our api key in both controller and vcr fixture files

Type of change

  • fix
  • feat
  • test
  • refactor
  • docs

Checklist

  • code has been self reviewed
  • code runs without any errors
  • thorough testing has been implemented if adding feature
  • all tests pass

Thanks!

elizakeating and others added 30 commits October 10, 2023 20:19
… to prevent errors, add order expect statements
Co-authored-by: Antoine Aube <55931218+Antoine-Aube@users.noreply.github.com>
Co-authored-by: Antoine Aube <55931218+Antoine-Aube@users.noreply.github.com>
Co-authored-by: Antoine Aube <55931218+Antoine-Aube@users.noreply.github.com>
Co-authored-by: Antoine Aube <55931218+Antoine-Aube@users.noreply.github.com>
Co-authored-by: Antoine Aube <55931218+Antoine-Aube@users.noreply.github.com>
Co-authored-by: Antoine Aube <55931218+Antoine-Aube@users.noreply.github.com>
Co-authored-by: Antoine Aube <55931218+Antoine-Aube@users.noreply.github.com>
Co-authored-by: Antoine Aube <55931218+Antoine-Aube@users.noreply.github.com>
Co-authored-by: Antoine Aube <55931218+Antoine-Aube@users.noreply.github.com>
Co-authored-by: Antoine Aube <55931218+Antoine-Aube@users.noreply.github.com>
This was linked to issues Oct 11, 2023
@Antoine-Aube Antoine-Aube merged commit a7de63d into main Oct 11, 2023
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.

Movie Details Page Movie Results Page Discover Movies Page
2 participants