Skip to content
This repository has been archived by the owner on Feb 9, 2023. It is now read-only.

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

Open
nejatcanunlu opened this issue Feb 8, 2023 · 8 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@nejatcanunlu
Copy link

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: acikyazilimagi/deprem-yardim-frontend#405 (comment)

@nejatcanunlu
Copy link
Author

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 nejatcanunlu changed the title Client'ı hızlandırmak için yeni marker data endpointleri afetharita client'ı hızlandırmak için yeni marker data endpointleri Feb 8, 2023
@cdagli
Copy link

cdagli commented Feb 8, 2023

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.

@nejatcanunlu
Copy link
Author

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.

@cdagli
Copy link

cdagli commented Feb 8, 2023

@okankaraduman
Copy link
Collaborator

okankaraduman commented Feb 8, 2023

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

#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 okankaraduman added enhancement New feature or request help wanted Extra attention is needed labels Feb 8, 2023
@serhhan
Copy link

serhhan commented Feb 8, 2023

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

Screenshot 2023-02-08 at 20 59 58

@serhhan
Copy link

serhhan commented Feb 8, 2023

@okankaraduman

@serhhan
Copy link

serhhan commented Feb 8, 2023

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants