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

get_locations_without_deleted_tweet method for #64

Open yarliganfatih opened this issue 2 years ago • 1 comments

Tam test edemedim henüz fakat uygun görülürse küçük bir code review ile fonksiyonu LocationViewSet'de çağırmamız yeterli olacaktır.

Hatta daha da geliştirilip veritabanında statik olarak is_deleted sütunuyla tutulabilir.

is_deleted=models.BooleanField(default=False)

Başlangıçta false olarak tutulan değer api'dan her istek çağırıldığında (ya da cron jobs ile periyodik olarak) eğer false ise api.twitter.com'dan kontrol edilecek. Ta ki tweet'in silindiği saptanana kadar. değer true'ya döndükten sonra bir daha kontrol edilmeyecek.

veritabanında tutacağımız is_deleted=true verileri ile haritada silinmişleri göster/gösterme seçeneği ayarlanabilir. Böylece haritada kalabalık veri yığınına sebep olmayacak. Ama asıl önemli olan ilerleyen zamanlarda silinmiş tweetlerin de asparagas olup olmadığının tespit edilmesini sağlayabilir.

yarliganfatih avatar Feb 07 '23 20:02 yarliganfatih

Backend architecture'ı değiştiği için bu değişiklik yapılamaz. @yarliganfatih

okankaraduman avatar Feb 08 '23 00:02 okankaraduman