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

Blake Sergesketter, Xander Hendry #96

Open
wants to merge 84 commits into
base: main
Choose a base branch
from

Conversation

XanderHendry
Copy link

We weren't able to get all the way through the project, we got up to creating a new viewing party, but couldn't get our route to create the viewing party and the connecting objects between viewing parties and invited users.

XanderHendry and others added 30 commits November 28, 2023 17:32
Styling corrections made to application according to rubocop.
…model tests cover validation, registration tests cover presence of content, and functionality of form, including sad path
…ion. All testing passes. Addition of model and model testing of a user for feature testing of existing users emails.
…ontent on /register page is tested, including sad path
refactor routes in root welcome page to make all tests passing.
…ting- testing of presence of a link to the home page. All tests passing.
…age it is, a button to discovering movies (no route yet), and a section that lists viewing parties.
coverage files aren't being ignored by git, even though it's been added to .gitignore
coverage files aren't being ignored
XanderHendry and others added 30 commits December 1, 2023 16:37
…s and search movies now redirect to the same page
Good work on breaking up the service into some helper methods. Nice use of SRP going on there!
…l of response from movies_controller- show action.
…esting of formatted_runtime. All testing passes.
no coverage in ginthub repo
…lude used routes, ensure all tests are passing"
…st creation as well as viewing_party creation
Start of testing a users dahsboard for viewing party. Testing incompl…
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