Skip to content

Reservation junction table, eager loading of listing reservations#42

Merged
Prakhar896 merged 5 commits intomainfrom
prakhar
Jul 8, 2024
Merged

Reservation junction table, eager loading of listing reservations#42
Prakhar896 merged 5 commits intomainfrom
prakhar

Conversation

@Prakhar896
Copy link
Copy Markdown
Contributor

@Prakhar896 Prakhar896 commented Jul 8, 2024

Overview of changes:

  • After much debugging, fixed and aligned associations between FoodListing, Guest and Reservation. Worked on N:M association where each FoodListing has many Guests through Reservation and each Guest has many FoodListings through Reservation.
  • Changed all listings to retrieve only published listings
  • Changed getListing endpoint to eagerly load Guest model which includes a Reservation object

Sensitive changes:

  • Changes made to FoodListing, Guest and Reservation models

@Prakhar896 Prakhar896 added the enhancement New feature or request label Jul 8, 2024
Prakhar896
Prakhar896 commented Jul 8, 2024
View reviewed changes
Copy link
Copy Markdown
Contributor Author

@Prakhar896 Prakhar896 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Prakhar896 Prakhar896 merged commit 66d97c4 into main Jul 8, 2024
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

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Prakhar896