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

Issue 93: Add ListMovie table #102

Merged
merged 1 commit into from
Jun 18, 2024
Merged

Issue 93: Add ListMovie table #102

merged 1 commit into from
Jun 18, 2024

Conversation

chrispinkney
Copy link
Member

Issue

Closes #93

Type of Change

  • Bugfix: Change which fixes an issue
  • New Feature: Change which adds functionality
  • Documentation Update: Change which improves documentation
  • UI: Change which improves UI

Description

This PR adds a new table to our db, the ListMovie bridging table between... list and movie.

It was created to allow for list and movie associations, specifically so lists can be sorted by association date (createdAt) when returned.

Testing the PR

Migrate, Generate, Run, create a new list, add movies to it, get movies.

Checklist

  • Quality: This PR builds and passes the respective npm run test and works locally
  • Tests: This PR includes thorough tests
  • Screenshots: This PR includes screenshots or GIFs of the changes made or an explanation of why it does not (if applicable)
  • Documentation: This PR includes updated/added documentation to user exposed functionality or configuration variables are added/changed or an explanation of why it does not(if applicable)

@chrispinkney chrispinkney added the api API related issue label Jun 10, 2024
@chrispinkney chrispinkney requested a review from manekenpix June 10, 2024 14:44
@chrispinkney chrispinkney self-assigned this Jun 10, 2024
@chrispinkney chrispinkney requested a review from manekenpix June 11, 2024 00:19
@manekenpix manekenpix changed the title Issue 93: Add ListMovie bridging table Issue 93: Add ListMovie table Jun 11, 2024
@chrispinkney chrispinkney requested a review from manekenpix June 14, 2024 16:13
@chrispinkney chrispinkney merged commit 0a95e2b into dev Jun 18, 2024
7 checks passed
@chrispinkney chrispinkney deleted the issue-93 branch June 18, 2024 01:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api API related issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants