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

Feature/models association #17

Merged
merged 25 commits into from
Feb 15, 2024
Merged

Feature/models association #17

merged 25 commits into from
Feb 15, 2024

Conversation

M-Anwar-Hussaini
Copy link
Collaborator

Summary of this PR:

  • Create associations between all relevant models.
  • Add some validations to the model attributes.
  • No linters errors

fmanimashaun and others added 13 commits February 15, 2024 22:35
@fmanimashaun
Update Gemfile lock
80420a9
@fmanimashaun
Update user model
2156c9f
@Estete9
Refactor car model so it has one to one association with car_detail a…
f6426bb 
…nd so it accepts nested attributes from car_detail
@Estete9
Refactor car_details model with more validations
f0c88fb
@Estete9
Refactor engine_details model with more validations
d493e31
@Estete9
Refactor reservation model with more validations
e451ede
@Estete9
Refactor city model with more validations
a6cc589
@Estete9
Merge branch 'feature/models-association' of https://github.com/fmani…
0cd52b5 
…mashaun/book-car-api into feature/models-association
@Estete9
Merge branch 'feature/models-association' of https://github.com/fmani…
ed9028b 
…mashaun/book-car-api into feature/models-association
@Estete9
Merge branch 'feature/models-association' of https://github.com/fmani…
d037d50 
…mashaun/book-car-api into feature/models-association
@Estete9
Add maximum length to car model\
01e21a1
@Estete9
Add maximum length to city model
2359282
@fmanimashaun
Minor adjustment to model association validation
8c316b2
@fmanimashaun fmanimashaun merged commit 80f577d into dev Feb 15, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fmanimashaun fmanimashaun Awaiting requested review from fmanimashaun

@Estete9 Estete9 Awaiting requested review from Estete9

@simplegoose simplegoose Awaiting requested review from simplegoose

@tomasesquivelgc tomasesquivelgc Awaiting requested review from tomasesquivelgc

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@M-Anwar-Hussaini @fmanimashaun @Estete9