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

feat: Frontend harita markerlarının yeni response modeline göre işlenmesi #405

Closed
khnbzkrt opened this issue Feb 8, 2023 · 8 comments
Labels
approved pm approved this issue emergency very urgent, priority issue help wanted need help on this

Comments

@khnbzkrt
Copy link
Contributor

khnbzkrt commented Feb 8, 2023

İsteğiniz şeyi tanımlayın

Frontend tarafındaki markerlar, backend tarafından gelen yeni response modeline göre düzeltilmelidir.

Değerlendirdiğiniz alternatifleri tanımlayın

Bulunmamaktadır

Ek Bağlam

Endpoint : /feeds/areas?ne_lat={ne_lat}&ne_lng={ne_lng}&sw_lat={sw_lat}&sw_lng={sw_lng}

Response Model :
{ "id":146, "count":5, "loc": [ 37.5859946, 36.9637805 ] }

@eraygundogmus eraygundogmus added approved pm approved this issue emergency very urgent, priority issue labels Feb 8, 2023
@cdagli
Copy link
Contributor

cdagli commented Feb 8, 2023

@khnbzkrt Bunu mevcut API'a dokunmadan ayri bir endpoint veya versiyon uzerinde yapmaniz mumkun mudur? Bizim icin breaking change cunku.

Backend issuesi hangisidir?

@eraygundogmus eraygundogmus added the help wanted need help on this label Feb 8, 2023
@nejatcanunlu
Copy link
Contributor

nejatcanunlu commented Feb 8, 2023

Son durumda konuşulan ihtiyaç:

ikisine de ?ne_lat={ne_lat}&ne_lng={ne_lng}&sw_lat={sw_lat}&sw_lng={sw_lng} gibi get parametreleriyle gelinecek;

1- Geniş zoomda yalnızca markerlerı görüntüleyebilmek için, response model'da yalnızca latitude, longitude ve pointID dönen bir özet endpoint

2- Dar zoomda, birinci maddedeki dataya ek olarak, adres, tweet datası v.s. detay datayı dönecek ayrı bir endpoint

Tek endpointe ekstra field ile gelip ayrımı da yapılabilir, hiç önemli değil, detayını bilmemiz yeterli.

Breaking change olacağı için yeni endpoint URL'lerde koşmaları önemli.

Backend reposuna konu ile ilgili açılan issue: acikyazilimagi/deprem-yardim-backend#116

@serhhan
Copy link
Contributor

serhhan commented Feb 8, 2023

Backend issue'sini inceledim, yapılan kısmındaki parametreleri postmanden denediğimde response değişmiyor.

Screenshot 2023-02-08 at 21 06 56

@serhhan
Copy link
Contributor

serhhan commented Feb 8, 2023

Screenshot 2023-02-08 at 21 07 53

@cdagli
Copy link
Contributor

cdagli commented Feb 8, 2023

@serhanelmali Bir de normal unix timestamp deneyebilir misin? Field adi timestamp ise belki de date olmayabilir?

Aksi halde https://github.com/acikkaynak/deprem-yardim-backend'e bug acabiliriz.

Bu arada bu timestamp icin ayri issue var FE tarafinda: #471

@goktugs
Copy link

goktugs commented Feb 8, 2023

@cdagli Selamlar. Unix timestamp ile denedim calısmadı. Gercekten işe yarayabilecek bir özellik. Yanlıs bakmadıysam BE tarafındada bu konu hakkında bir aksiyon alınmamıs. Bug acarsanız sevinirim

@serhhan
Copy link
Contributor

serhhan commented Feb 8, 2023

@goktugs ++

@cdagli
Copy link
Contributor

cdagli commented Feb 8, 2023

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved pm approved this issue emergency very urgent, priority issue help wanted need help on this
Projects
None yet
Development

No branches or pull requests

6 participants