deprem-yardim-backend icon indicating copy to clipboard operation
deprem-yardim-backend copied to clipboard

afetharita client'ı hızlandırmak için yeni marker data endpointleri

Open nejatcanunlu opened this issue 2 years ago • 8 comments

Merhabalar, frontend tarafından geliyorum;

afetharita.com'daki haritanın ilk yüklenmesini ve kullanımını hızlandırmak için iki yeni endpointe ihtiyaç var:

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

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

İlgili frontend issue: https://github.com/acikkaynak/deprem-yardim-frontend/issues/405#issuecomment-1422299793

nejatcanunlu avatar Feb 08 '23 09:02 nejatcanunlu

Anladığımız kadarıyla backend tarafında bu konuda çalışılmış, hatta hazır gibi ama başka bir sorundan merge bekliyormuş, konu kaybolmasın diye not düşmek istedik.

nejatcanunlu avatar Feb 08 '23 09:02 nejatcanunlu

2 ayri endpoint yerine fields de eklenebilir query'e. Eger gonderirsek o fieldlari, gondermezsek hepsini. Ya da all gibi bir keyword de olabilir, tam convention ne bilmiyorum.

Ama implementation yapildiysa daha sonra degerlendirilebilir bu fikir.

cdagli avatar Feb 08 '23 09:02 cdagli

2 ayri endpoint yerine fields de eklenebilir query'e. Eger gonderirsek o fieldlari, gondermezsek hepsini. Ya da all gibi bir keyword de olabilir, tam convention ne bilmiyorum.

Ama implementation yapildiysa daha sonra degerlendirilebilir bu fikir.

O da olur, hiç fark etmez. Hal-i hazırda yayındaki endpointten farklı URL'de yaşaması yeterli, ekstra fieldle gelinecek tek veya iki endpoint olabilir.

nejatcanunlu avatar Feb 08 '23 09:02 nejatcanunlu

Backend PR'i su galiba https://github.com/acikkaynak/deprem-yardim-backend/pull/113/files

cdagli avatar Feb 08 '23 09:02 cdagli

Arkadaşlar selam, @cdagli , @nejatcanunlu Coordinate filtreleme konusunda değil ama optimizasyon adına bir iş yapılmıştı.

https://github.com/acikkaynak/deprem-yardim-backend/pull/90

Frontend'den bir arkadaşın önerisiyle geliştirildi ve merge edildi.

?timestam__gte=2023-02-08 ?timestam__lte=2023-02-08 queryleriyle o tarihe kadar ve oradan sonra olan verileri getirebilirsiniz teoride. Doğru düzgün test yapamadık.

Frontend arkadaşın(Ogün Baysal) önerisi şurdan alınabilir:

https://discord.com/channels/1072074800622739476/1072074878452240384/1072605749055860787

okankaraduman avatar Feb 08 '23 13:02 okankaraduman

Selam postman tarafında test ettim fakat sonuçlarda bir değişme gözlemleyemedim.

Screenshot 2023-02-08 at 20 59 58

serhhan avatar Feb 08 '23 18:02 serhhan

@okankaraduman

serhhan avatar Feb 08 '23 18:02 serhhan

timestamp, __ ve _ li versiyonlarını da denedim.

serhhan avatar Feb 08 '23 18:02 serhhan