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

remove adress from collections (models) and fix getHotelsByCity (effort: 2) #68

Open
pdybowski opened this issue Mar 2, 2021 · 0 comments
Open

remove adress from collections (models) and fix getHotelsByCity (effort: 2) #68

pdybowski opened this issue Mar 2, 2021 · 0 comments
Assignees
@hoolek77
Labels
bug Something isn't working priority HIGH

Comments

@pdybowski
Copy link
Owner

pdybowski commented Mar 2, 2021 •
edited
Loading

We shouldnt have collection with adresses, just store adress object inside hotel object
so we need only schema and validation method

image

Once this is done cgane getHotelsByCity in services to:
image
@pdybowski pdybowski added bug Something isn't working priority HIGH labels Mar 2, 2021
@pdybowski pdybowski changed the title remove adress from collections (models) remove adress from collections (models) and fix getHotelsByCity Mar 2, 2021
@pdybowski pdybowski changed the title remove adress from collections (models) and fix getHotelsByCity remove adress from collections (models) and fix getHotelsByCity (effort: 2) Mar 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hoolek77 hoolek77

Labels
bug Something isn't working priority HIGH
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pdybowski @hoolek77