Code

Joop and Reed Turn-in#82

Code

Open

JoopStark wants to merge 100 commits intoturingschool-examples:mainturingschool-examples/viewing_party_lite_7:mainfrom JoopStark:mainJoopStark/viewing_party_lite_7:mainCopy head branch name to clipboard

+16,438-7

Commits

Commits on Oct 9, 2023

• setup: complete

  JoopStark

  committed
  5ffa45a

• add: routes

  JoopStark

  committed
  0b671b3

• Merge pull request #8 from JoopStark/joop-landing

  Show description for abf0dc4
  reedhillmar

  authored
  abf0dc4

• setup: adds gems

  reedhillmar

  committed
  d6aa92a

Commits on Oct 10, 2023

• setup: schema setup

  reedhillmar

  committed
  6bef21a

• setup: more validations

  reedhillmar

  committed
  667acf2

• setup: viewing party validations

  reedhillmar

  committed
  7559de2

• test: adds model tests

  reedhillmar

  committed
  be95b30

• Merge pull request #9 from JoopStark/reed_setup

  Show description for 028904b
  JoopStark

  authored
  028904b

• user views

  JoopStark

  committed
  9444e4e

• Merge branch 'main' of github.com:JoopStark/viewing_party_lite_7 into joop-landing

  JoopStark

  committed
  1eeef9f

• Merge pull request #10 from JoopStark/joop-landing

  Show description for 519a1aa
  reedhillmar

  authored
  519a1aa

• feat tests: welcome index page; header, new user button, current user email links

  JoopStark

  committed
  aa9d11e

• Merge branch 'main' into joop-landing

  JoopStark

  authored
  f229692

• feat test: user show page has header, discover button, and table with viewing party attributes

  JoopStark

  committed
  b3fd397

• Merge branch 'main' of github.com:JoopStark/viewing_party_lite_7 into joop-landing

  JoopStark

  committed
  5e4387b

• Merge branch 'joop-landing' of github.com:JoopStark/viewing_party_lite_7 into joop-landing

  JoopStark

  committed
  d5166bb

• Merge pull request #11 from JoopStark/joop-landing

  Show description for 712a008
  reedhillmar

  authored
  712a008

• feat test bug fixes: changed buttons from post to get

  JoopStark

  committed
  314bbc2

• schema: add image_path column to viewing_parties

  JoopStark

  committed
  7f42d38

• Merge pull request #22 from JoopStark/joop-landing

  Show description for 7cc1e26
  reedhillmar

  authored
  7cc1e26

• feat: adds discover page and the start of search form

  reedhillmar

  committed
  20b2c28

Commits on Oct 11, 2023

• Merge pull request #23 from JoopStark/reed_discover

  Show description for b1f79c0
  JoopStark

  authored
  b1f79c0

• feat: movies index controller, changed schema

  JoopStark

  committed
  c4472fc

• Merge pull request #24 from JoopStark/joop-spike

  Show description for 3666c3f
  reedhillmar

  authored
  3666c3f

• fix schema and test helper

  JoopStark

  committed
  911998b

• Merge pull request #25 from JoopStark/joop-movies_index

  Show description for 9bcc25e
  reedhillmar

  authored
  9bcc25e

• gems and vcr setup

  JoopStark

  committed
  9077e8a

• test: adds test for basic page setup

  reedhillmar

  committed
  0f6dc5d

• Merge pull request #27 from JoopStark/movie_details

  Show description for 8acfae3
  JoopStark

  authored
  8acfae3

• Merge branch 'main' into joop-movies_index

  reedhillmar

  authored
  17a279e

• Merge pull request #26 from JoopStark/joop-movies_index

  Show description for 455cd75
  reedhillmar

  authored
  455cd75

• test: test code for create view party button

  reedhillmar

  committed
  66da57e

• feat: adds create viewing party button functionality to movie details page

  reedhillmar

  committed
  fadba91

• feat: adds movie details to the movie details page

  reedhillmar

  committed
  04b4a0b

• test: adds test code for runtime and genres in movie details spec

  reedhillmar

  committed
  95720c5

• feat: adds functionality for vote average, runtime, and genres to movie details page

  reedhillmar

  committed
  5d942fb

• feat test: movie index can display top 20 and keyword search

  JoopStark

  committed
  78175e6

• feat: movie index can display top 20 and keyword search

  JoopStark

  committed
  73657b9

• test/feat: adds test and functionality for displaying the movie summary on the movie details page

  reedhillmar

  committed
  af37b20

• Merge pull request #28 from JoopStark/joop-movies_index

  Show description for ed7bf7f
  reedhillmar

  authored
  ed7bf7f

• test/feat: adds test code and functionality for showing cast members of a movie and their characters to the movie details page

  reedhillmar

  committed
  2881183

Commits on Oct 12, 2023

• test: test code for total number of reviews on the movie details page

  reedhillmar

  committed
  91fdc15

• feat: shows total reviews on the movie details page

  reedhillmar

  committed
  7ddcfb8

• test: adds test code for displaying reviews with their author and rating on the movie details page

  reedhillmar

  committed
  5d7c4b4

• feat: adds functionality to show a review, its author, and the review rating on the movie details page

  reedhillmar

  committed
  04e0513

• poros: movie

  JoopStark

  committed
  ecdd351

• feat test: User show

  JoopStark

  committed
  bd7946b

• feat: User show

  JoopStark

  committed
  ad48dcd

• refactor: users show

  JoopStark

  committed
  5236dbb

• Merge pull request #29 from JoopStark/movie_details

  Show description for 0ebae72
  JoopStark

  authored
  0ebae72

• Merge pull request #30 from JoopStark/joop-users_show

  Show description for 296d0c2
  reedhillmar

  authored
  296d0c2

• feat: runtime in MovieSearch poro

  JoopStark

  committed
  21040a7

• feat: party new

  JoopStark

  committed
  ac1f345

• feat: Viewing party new, multiple select invites and date and time params

  JoopStark

  committed
  4c66484

• feat test: New User with sad paths

  JoopStark

  committed
  5eef78f

• Merge pull request #31 from JoopStark/joop-party_new

  Show description for 230c81e
  reedhillmar

  authored
  230c81e

• bug: remove bug spec naming caused error in vcr

  JoopStark

  committed
  43cdb00

• Merge pull request #32 from JoopStark/joop-party_new_spec

  Show description for 08171ef
  reedhillmar

  authored
  08171ef

• refactor: adds movie service and facade, refactors show page to use

  reedhillmar

  committed
  fa6c9c2

Commits on Oct 13, 2023

• Merge pull request #33 from JoopStark/services

  Show description for 3950284
  JoopStark

  authored
  3950284

• test: sad path testing for viewing party new

  JoopStark

  committed
  776c5f7

• Merge pull request #34 from JoopStark/joop-party_new

  Show description for 4c62446
  reedhillmar

  authored
  4c62446

• refactor: more facade and service refactoring for movies index

  reedhillmar

  committed
  34a47bc

• Merge pull request #35 from JoopStark/services

  Show description for 8a7dc6a
  JoopStark

  authored
  8a7dc6a

• refactor: add cast poro and update movie facade and show page to utilize

  reedhillmar

  committed
  ca7e447

• poros test: Movie exist and has attribures

  JoopStark

  committed
  300e9ee

• poros: movie and views fix

  JoopStark

  committed
  2a47dcd

• refactor: reviews poro and update movies facade and show page to utilize

  reedhillmar

  committed
  6405f59

• refactory remove bad poros

  JoopStark

  committed
  ffc6c8b

• fix: sad path created nil rounding problem with movie.vote average

  JoopStark

  committed
  2cd453b

• refactor: movie.vote_average

  JoopStark

  committed
  5011603

• Merge pull request #37 from JoopStark/reed_poro

  Show description for aa9ae81
  JoopStark

  authored
  aa9ae81

• Merge branch 'main' into joop-movie_poro

  reedhillmar

  authored
  292c1ec

• Merge pull request #36 from JoopStark/joop-movie_poro

  Show description for d06c12e
  reedhillmar

  authored
  d06c12e

• service test: Movie Service

  JoopStark

  committed
  3e19695

• test: adds testing for POROs and fixes welcome index test

  reedhillmar

  committed
  a1aa0d6

• Merge pull request #38 from JoopStark/reed_tests

  Show description for a094903
  JoopStark

  authored
  a094903

• create facade test

  JoopStark

  committed
  4fd7179

• Merge branch 'main' of github.com:JoopStark/viewing_party_lite_7 into joop-movie_poro

  JoopStark

  committed
  b34778c

• service test: MoviesService

  JoopStark

  committed
  3055a4a

• refactor: rubocoped facade spec

  JoopStark

  committed
  48c6805

• fix: rubocop fixes

  reedhillmar

  committed
  47fa99e

• refactor: rubocoped specs: models, facade, services

  JoopStark

  committed
  d097713

• fix: rubocop pulled me over for double quotes and issued me a citation

  reedhillmar

  committed
  6499141

• Merge pull request #39 from JoopStark/reed_rubocop

  Show description for 02b1992
  JoopStark

  authored
  02b1992

• merge conflict fix, took incoming

  JoopStark

  committed
  7cce921

• cleeeeean

  JoopStark

  committed
  fed8234

• Merge pull request #40 from JoopStark/joop-movie_poro

  Show description for 54a46ee
  JoopStark

  authored
  54a46ee

• refactored user show page

  JoopStark

  committed
  1ec1b8d

• refactor/test/fix: small tweaks to cover bases for all user stories, everything works

  reedhillmar

  committed
  faa1589

• Merge pull request #41 from JoopStark/reed_rubocop

  Show description for 27e3813
  JoopStark

  authored
  27e3813

• Merge branch 'main' of github.com:JoopStark/viewing_party_lite_7 into joop-movie_poro

  JoopStark

  committed
  a36d01f

• Merge pull request #42 from JoopStark/joop-movie_poro

  Show description for e97385e
  reedhillmar

  authored
  e97385e

Commits on Oct 14, 2023

• refactory: pagentry

  JoopStark

  committed
  28efdb0

• Merge pull request #43 from JoopStark/joop-demo

  Show description for 2df584f
  reedhillmar

  authored
  2df584f

Commits on Oct 24, 2023

• test feat: authentication

  JoopStark

  committed
  de082f1

• Merge pull request #44 from JoopStark/joop-authentication

  Show description for ac2eec7
  JoopStark

  authored
  ac2eec7

• removed pry

  JoopStark

  committed
  e845db8

Commits on Oct 25, 2023

• authorization

  JoopStark

  committed
  22da37a