Skip to content

feat: added /feeds/busy endpoint for retrieving busy locations (lat, lng) #46

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

Closed
wants to merge 7 commits into from

Conversation

younver
Copy link

@younver younver commented Feb 9, 2023

Bu endpoint AddressAPI'da çakışmalarını engellemek için ve Frontend'de UX'i arttırmak için gerekli yoğun lokasyon bilgisini sağlamaktadır. /feeds/busy haritadaki entry'lerin ortalamasını alarak yoğun olan lokasyonu (latitude ve longitude formatında) bulur ve döndürür. Sistemin diğer ayakları bu bilgiyi kullanarak yoğun olan adrese yönlendirmek ve kullanıcıya yoğun olan bölgeyi göstermek gibi işlemleri gerçekleştirebilir.

discord: @Kitty~~#0705

closes acikyazilimagi/deprem-yardim-frontend#494
closes acikyazilimagi/deprem-yardim-backend#115
her ne kadar arşivlenmiş proje olsa da önemli bir hata olduğunu düşünüyorum.

PR açmadan önce dikkat edilmesi gerekenler

  • [ X] Kendi kodumu inceledim, review ettim.
  • Eğer core bir feature ise, detaylı testler yaptım.

PR açma kuralları

  • [X ] Approved tagine sahip bir issue için PR açılmalıdır. Aksi takdirde PR reddedilecektir.
  • [X ] İlgili issue numarası: PR ile ilgili issue numarası, PR başlığının başına, prefix sonrası parantez içinde # ile eklenmelidir. "prefix(#issue_number): PR başlığı" şeklinde bir başlık kullanılabilir.
  • [X ] Açıklayıcı ve anlaşılır bir başlık: PR başlığı, değişikliklerin niteliklerini ve amaçlarını açık bir şekilde tanımlamalıdır. Pr başlığı, PR açıldığında görüntülenen ilk şey olmalıdır. Ve semantik commit kurallarını takip etmelidir. Örneğin "docs(#issueId): Add README.md" şeklinde bir başlık kullanılabilir.
  • [X ] İlgili dosya seçimi: Sadece ilgili dosyalara dokunulmalı ve başka dosyaların etkilenmemesi sağlanmalıdır.
  • [ X] Format ve Lint uygunluğu: Kod, belirli bir format standardına uygun hale getirilmeli ve lint kurallarına göre incelenmelidir.
  • [X ] Temiz commit geçmişi: Değişikliklerin yapıldığı commitler, anlaşılır ve düzenli olmalıdır.
  • [X ] İş tamamlandığında PR açılması: PR, iş tamamlandığında açılmalı ve diğer takım üyeleri tarafından incelenmesi için gönderilmelidir.

Değişiklikler

  • [ X] Yeni bir özellik (breaking change olmayan, yeni bir özellik ekleyen bir değişiklik)
  • Yeni bir refactor (breaking change olmayan, kodun okunabilirliğini veya performansını artıran bir değişiklik)
  • Breaking bir değişiklik
  • Dokümantasyon değişikliği

Test için bana yardımcı olabilecek birisi olursa çok sevinirim. Discorddan, buradan her yerden ulaşabilirsiniz.

@aysenur-aydin aysenur-aydin added the kurala-uygun-pr bu pr pr açma kurallarına uygundur label Feb 9, 2023
@mstrYoda mstrYoda added invalid yanlış açılmış issue veya pr, kurallarla uyumsuz wontfix üzerinde çalışılmayacak iş, iptal, istenmeyen geliştirme and removed kurala-uygun-pr bu pr pr açma kurallarına uygundur labels Feb 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid yanlış açılmış issue veya pr, kurallarla uyumsuz wontfix üzerinde çalışılmayacak iş, iptal, istenmeyen geliştirme
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Filter/Fix buyuksehir deprem raporlari bug: False position statement in similar addresses
3 participants