acikyazilimagi / deprem-yardim-frontend

release canditate: https://rc.afetharita.com/
https://afetharita.com/
Apache License 2.0
895 stars 283 forks source link

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

Closed khnbzkrt closed 1 year ago

khnbzkrt commented 1 year ago

İ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 ] }

cdagli commented 1 year ago

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

Backend issuesi hangisidir?

nejatcanunlu commented 1 year ago

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: https://github.com/acikkaynak/deprem-yardim-backend/issues/116

serhhan commented 1 year ago

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 commented 1 year ago

Screenshot 2023-02-08 at 21 07 53

cdagli commented 1 year ago

@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: https://github.com/acikkaynak/deprem-yardim-frontend/issues/471

goktugs commented 1 year ago

@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 commented 1 year ago

@goktugs ++

cdagli commented 1 year ago

Related: https://github.com/acikkaynak/deprem-yardim-backend/issues/138