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

Silinmiş tweetler #64

Open
merterdir opened this issue Feb 7, 2023 · 4 comments
Open

Silinmiş tweetler #64

merterdir opened this issue Feb 7, 2023 · 4 comments
Labels
enhancement New feature or request

Comments

@merterdir
Copy link

şu anda çok fazla kaynağı silinmiş tweet var haritada. acaba backend'de çözülen vakalar için bir planlama var mı? (afad veya vatandaş tarafından kurtarılanların / yanlış bilgilerin silinmesi için)

otomatize etmek için bir öneri de kaynak eğer tweet ise backend'de otomatik kontrol edip silinen tweetleri haritadan filtrelemek olabilir.

@okankaraduman
Copy link
Collaborator

Bu problemle alakalı olarak 2 tane adım söz konusu diyebiliriz.

1- Eski verilerin temizlenmesi gerekiyor:
Bunun için databasedeki tweets table'ındaki name ve tweet_id parametreleri ile generate edilen https://twitter.com/{name}/status/{tweet_id} ziyaret edilerek status_code != 2XX değilse entry database'den silinebilir.
2- Frontend'de eğer tweet bulunamıyorsa tetiklenecek bir error boundary yapısı eklenilebilir. https://nextjs.org/docs/advanced-features/error-handling

yarliganfatih added a commit to yarliganfatih/deprem-yardim-backend that referenced this issue Feb 7, 2023
@yarliganfatih
Copy link

#79

Burada da bahsettiğim gibi, bence bu verilerin veritabanından silinmemesi daha iyi olur. API tarafında gerekli filtrelemeleri yaparak haritadaki kalabalık görünen veri yığınının bir nebze önüne geçilebilir.

@UltraInstinct0x
Copy link

Aksine silinmiş tweetler daha sonradan filtrelenebileceklerdir ve bu veriyi ileriki safhalarda daha iyi filtreledikçe daha efektif kullanabiliriz.

@okankaraduman
Copy link
Collaborator

Soft delete yapılabilir en azından @gokhansarapevi

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

No branches or pull requests

4 participants