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

Bir ile ait tüm veterinerlerin listesi #8

Closed
okandavut opened this issue Apr 25, 2018 · 3 comments
Closed

Bir ile ait tüm veterinerlerin listesi #8

okandavut opened this issue Apr 25, 2018 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@okandavut
Copy link

okandavut commented Apr 25, 2018

Selamlar,
Öncelikle api çok kullanışlı elinize sağlık.

Bir ile ait listelidiğimde şu url ile sınırlı sayıda veteriner geliyor : http://veterinaries.enderahmetyurt.com/search?city=izmir

Ancak şu şekillerde aradığımda her sayfada farklı veterinerler geliyor :
http://veterinaries.enderahmetyurt.com/search?city=izmir&page=1
http://veterinaries.enderahmetyurt.com/search?city=izmir&page=2
http://veterinaries.enderahmetyurt.com/search?city=izmir&page=3

Tüm veteriner listesinin tek json olarak geldiği yani atıyorum Izmir' e ait tüm veterinerleri listeyen url eklemesi yapılabilir. Ya da ben gözümden kaçırdım 😄
Ben bu url ' in ( http://veterinaries.enderahmetyurt.com/search?city=izmir ) tüm izmirdeki veterinerleri döndüreceğini düşünmüştüm.

Teşekkürler,
Saygılarımla.

@enderahmetyurt
Copy link
Owner

@okandavut haklısın. http://veterinaries.enderahmetyurt.com/search?city=izmir ve http://veterinaries.enderahmetyurt.com/search?city=izmir&page=1 aynı geliyor. page'siz olunca tümünün gelmesi gerek. @peyloride inceleyebilir misin?

@peyloride
Copy link
Collaborator

@okandavut Öneri için teşekkürler, bahsettiğin şekilde düzenledim. @enderahmetyurt PR'ı inceleyebilir misin?

enderahmetyurt added a commit that referenced this issue Apr 26, 2018
#8 - Show all vets if no params given in search endpoint
@enderahmetyurt
Copy link
Owner

Eline sağlık @peyloride Şimdi deploy edemiyorum @okandavut edince haber ederim.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants