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.

Sign up for GitHub

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

Jump to bottom

Kelsie Porter & Thomas Casady #207

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

Kelsie Porter & Thomas Casady #207

wants to merge 147 commits into from

Conversation

KelsiePorter
Copy link

No description provided.

Tscasady and others added 30 commits January 30, 2023 18:17
@Tscasady
Create pull_request_template.md
1d3cafd
@KelsiePorter
Merge pull request #1 from KelsiePorter/Tscasady-patch-1
5634638 
Create pull_request_template.md
@Tscasady
test template
0080ca6
@Tscasady
Setup gems and User model.
143e488
@Tscasady
Add coverage to gitignore.
c5c49ed
@Tscasady
Add relations to user model.
05ddd63
@Tscasady
Add relations to user and update schema.
5c936fe
@Tscasady
Add viewing_party_users model and update database.
4e06bf6
@Tscasady
Update user relations names for clarity.
33904a4
@KelsiePorter
Merge pull request #2 from KelsiePorter/setup
36ace10 
Setup
@KelsiePorter
Create welcome contoller, view, and routes
ef30f61
@Tscasady
Add test for user show page.
676e1d4
@Tscasady
Add route and controller for users.
4fdcc58
@KelsiePorter
Add faker for user name and email
47134e6
@KelsiePorter
Create list of existing users on landing page
7c759e9
@Tscasady
Add discover route to users.
56f6edf
@Tscasady
Update factories with better defaults.
b1ad268
@Tscasady
feat: Add functionality to display a user's viewing parties.
15c0613
@KelsiePorter
Create button to create a new user
f73541e
@KelsiePorter
Add nav bar home link to layouts
d7d572c
@KelsiePorter
Test for "home" link at top of page
1312bc8
@Tscasady
Remove redundant expect statement.
f217f8c
@KelsiePorter
Create user registration tests
eb9fc54
@Tscasady @KelsiePorter
Add route and controller for users.
0647b37
@Tscasady @KelsiePorter
feat: Add functionality to display a user's viewing parties.
bcac2ed
@Tscasady
Skips failing tests.
dde2908
@KelsiePorter
Merge pull request #3 from KelsiePorter/user/dashboard
cb79660 
User/dashboard
@KelsiePorter
Create register page and new user form
ca1e7f1
@KelsiePorter
Create validations and email uniqueness
2aba811
@KelsiePorter
Add nav bar and title for all views
43ad815
KelsiePorter and others added 30 commits February 2, 2023 20:47
@KelsiePorter
Create new viewing party form
77d3330
@KelsiePorter
Refactor
145134c
@Tscasady
Merge pull request #11 from KelsiePorter/new-viewing-party-page
31c8533 
New viewing party page
@Tscasady
Remove save_and_open_page.
80de142
@Tscasady
Add movie poro tests.
8679195
@Tscasady
Add facade test for movie method, add more simplecov filters.
0310712
@Tscasady
Add test for link to movie details page on user dashboard.
d1175cc
@Tscasady
Fix movie title link on user dashboard linking to movies instead of m…
e0eddb3 
…ovie_show.
@Tscasady
Center Existing Users on landing page.
22ee545
@Tscasady
Add viewing_party columns and center elements.
18a13d4
@Tscasady
Update movie show page styling.
c5d5b54
@KelsiePorter
Merge pull request #13 from KelsiePorter/styling-sweep
5ad077e 
Styling sweep
@KelsiePorter
Merge pull request #12 from KelsiePorter/fix/movie-link
d0da03b 
Fix/movie link
@Tscasady
test: Add test for facade movie building methods.
cc93d9d
@Tscasady
Refactor parse and movie to helpers, move movie building to facade.
aaa3132
@Tscasady
Add movie creation functionality.
d48c69a
@Tscasady
Add tests for service api calls.
0e6b61c
@Tscasady
Update facade test to include sample movie data.
bc9c1e5
@KelsiePorter
Merge pull request #14 from KelsiePorter/refactor/service-facade
f93455e 
Refactor/service facade
@Tscasady
Fix broken expectation and add test for new vps.
3f39485
@KelsiePorter
Merge pull request #15 from KelsiePorter/test/newvp
3a3e28d 
Fix broken expectation and add test for new vps.
@Tscasady
Implement structs instead of manipulating raw hashes in movie_show.
c0c43a2
@KelsiePorter
Merge pull request #16 from KelsiePorter/feature/openstruct
6a56a31 
Implement structs instead of manipulating raw hashes in movie_show.
@Tscasady
Update test to use struct.
f4096a3
@KelsiePorter
Merge pull request #17 from KelsiePorter/fix/struct
199b169 
Update test to use struct.
@Tscasady
Rubocop autocorrects.
043d410
@KelsiePorter
Merge pull request #18 from KelsiePorter/rubo/sweep
752d10a 
Rubocop autocorrects.
@KelsiePorter
Add authentication login for creating new user and user login
fcbb47f
@KelsiePorter
Updates to tests
40620f4
@KelsiePorter
Merge pull request #19 from KelsiePorter/authentication
1f56928 
Authentication
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@KelsiePorter @Tscasady