demir/pinmapper

Explore Pins around the world

pinmapper's Issues

Pinmapper Maps Api için web site kısıtlaması eklenmeli

Açıklama

Api'yi dışardan kullanıma kapatmak için web site kısıtlaması eklemek gerekiyor. Bu sayede kotanın istemsizce şişmesi engellenir.

İlgili kısıtlama: Application restrictions -> HTTP referrers (web sites)

Pin modelindeki privacy alanı kaldırılmalı

Bu tasarım önerisi bir problemle mi ilgili? Lütfen detaylıca açıklayınız

Pin modelinde şu an aktif kullanılmayan privacy alanı kaldırılmalı. Çünkü Pinmapper başlangıçta para kazanamayacak. Bu süreçte masraflar cepten karşılanacak. Pin'ler fotoğraf içerebileceği için ve konum servisini kullandığı için her Pin masraf demek. Bu yüzden private Pin'ler uygulama Pin havuzuna bir katkı sunmayacağından başlangıç aşamasında bu özelliği devre dışı bırakmak gerekiyor. İlerleyen dönemlerde Pinmapper maddi açıdan kendini döndürmeye başladığında privacy özelliği yeniden eklenebilir.

Sizin çözümünüz

Şimdilik privacy özelliği kaldırılmalı.

Pin oluştururken cropper'dan scroll ile zoom özelliği kaldırılmalı

Bu tasarım önerisi bir problemle mi ilgili? Lütfen detaylıca açıklayınız

Pin oluştururken imaj crop aşamasında yanlışlıkla scroll ile zoom yapılabiliyor. Bu durumda imaj küçülüyor ya da büyüyor ve crop işi zorlaşıyor.

Sizin çözümünüz

Crop sırasında scroll ile zoom özelliği kaldırılabilir

Ekran görüntüleri

Başkasının Panosu takip edilebilmeli

Açıklama

Kullanıcıların başkalarının Panolarını takip edebilmesi gerekiyor. Bunun için Panolara takip et veya takibi bırak butonları eklenmeli

Pin üzerindeki adrese tıklayınca location ekranı açılsın

Açıklama

PinComponent üzerinden ya da pin detay ekranı üzerinden yol tarifi alma ya da haritada konumu gösterme özelliği eklenmesi faydalı olur. Yol tarifi alma için Pin'in adres bilgisi üzerine tıklama özelliği eklenebilir. Bu bölüm component de yapılabilir.

Harita için ayrı bir ekran oluşturulabilir. Pin üzerindeki adres bilgisine tıklandığında bu ekran açılır. Ekranda ilgili pin haritada gösterilir. Bu pinle beraber yakındaki pinler ve sponsorlu pinler de gösterilebilir. Bu arada instagramda harita için sadece leaflet + openstreetmap kullanılmış.

Örnek:

Arama (Keşfet) ekranı eklenmeli

Açıklama

Kullanıcı, Pin ve Pano (gerekmeyebilir) aramasının yapılabileceği bir ekran gerekiyor. Bu sayede çok daha fazla Pin'e erişilebilir.

Notlar

PgSearch iyi bir tercih gibi görünüyor. Yeni bir şey eklemeden bu şekilde devam edilebilir.

Kaynaklar

• Arama için ransack gem kullanılabilr.
• Bu konuda dev.to (forem)'nun açtığı PR incelenebilir: forem/forem#5447
• dev.to (forem) search_controller_rb
• Elasticsearch kullanma: forem/forem#13606
• ransack vs searchkick: Building a search feature in Rails
• https://github.com/forem/forem/tree/main/app/services/search

Favicon eklenmeli

Açıklama

Favicon eklemek gerekiyor.

Kullanıcı takip özelliği eklenmeli

Açıklama

Kullanıcıların birbirini takip edebilmesi gerekiyor. Anasayfada takip edilen kullanıcıların pinleri listelenmeli.

Sınırlı input alanları için karakter sayıcı eklenmeli

Açıklama

Sınırlı input alanları için karakter sayıcı eklenmeli. Stimulus controller ile halledilebilir.

URL'e geçerli locale'ler dışında bir veri girildiğinde uygulama hata veriyor

Hatayı açıklayın

Locale olarak tr ya da en'den başka bir değer girildiğinde uygulama hata veriyor.

Beklenen davranış

Hata vermemesi gerekiyor. Bunun yerine varsayılan locale aktif edilmeli

Mevcut davranış

Uygulama hata veriyor

Ekran görüntüleri

Muhtemel çözüm

Hata:

Completed 500 Internal Server Error in 1ms (ActiveRecord: 0.0ms | Allocations: 175)
FATAL -- :
I18n::InvalidLocale (:ar is not a valid locale):
app/controllers/application_controller.rb:27:in `with_locale'

Bu bölüm düzenlenmeli

Hatayı tekrarlamak için takip edilmesi gereken adımlar

1. Herhangi bir sayfayı aç
2. Locale olarak tr ve en dışında bir değer gir

İş kayıtları ve PR'lar için şablon oluşturulmalı

İş kayıtlarını ve PR'ları belli bir düzende açmak için şablonlar oluşturulmalı

İlk iş kaydı 🎉

Profil fotoğrafının tam hali indirilememeli

Açıklama

Profile fotoğrafı kırpılmış gibi görünse de gerçekte öyle değil. Farklı kaydet ile tam hali indirilebiliyor. Bu pek doğru değil gibi.

Profil ekranına bio alanı eklenmeli

Açıklama

Profile ekranına kişisel bilgilerin yer alacağı bio (text) alanı eklenmeli. Kullanıcılar bu alana iletişim vb bilgilerini yazabilir. Örneğin mail adresi ya da sosyal medya adresleri.

Pano açıklama alanı eklenebilir

Açıklama

Pano açıklama alanına pano ile ilgili açıklama eklenebilmesi iyi olur. Zorunlu bir alan değil. Bu alan text olabilir. Pano büyük bir gezi için açılmışsa bu alanda gezinin değerlendirmesi yapılabilir. Fotoğraf bile eklenebilir.

Devise ile hesap oluşturduktan / onayladıktan sonra otomatik giriş yapmalı

Açıklama

Şu an otomatik giriş yapmıyor. Onayladıktan sonra giriş yapmak gerekiyor.

SmallPinComponent üzerinde panoya ekle/çıkar butonu olmalı

Açıklama

Pano üzerinde gösterilen Pin'leri panodan çıkarmak için Pin detay ekranına girmek gerekiyor. Bunun yerine bu ekranda bu işlemi yapabilmek gerekiyor. Bunun için SmallPinComponent üzerinde panoya ekle/çıkar butonun küçültülmüş hali eklenebilir.

Pin ekranları eklenmeli

Açıklama

Temel seviyede pin ekranları eklenmeli.

Oturum açan kullancılar;

• Pin oluşturabilmeli
• Kendi Pin'lerini düzenleyebilmeli
• Kendi Pin'lerini silebilmeli
• Diğer kullanıcıların pinlerini beğenebilmeli

Oturum açmamış kullanıcılar;

• Herkese açık Pin'leri görebilmeli

Pano detay ekranında bölümler başlığının görünürlüğü düzenlenmeli

Açıklama

Pano detay ekranındaki kullanıcı pano sahibi ise Bölümler başlığını bölüm ekle butonunu ve henüz bölüm eklemediniz bilgilendirme yazısını görsün. Pano sahibi değilse, bölüm var ise bölümler başlığını ve bölümleri görsün; bölüm yoksa, bölümle ilgili birşey görmesin.

Yasaklı usernameler belirlenmeli

Açıklama

Alınması yasak olan usernameler belirlenmeli ve alınmaları engellenmeli.

Pin içeriğine eklenen imajlar bir süre sonra siliniyor

Hatayı açıklayın

Pin içeriğine (active storage) eklenen imajlar bir süre sonra siliniyor.

Beklenen davranış

Silinmemesi gerekiyor.

Ekran görüntüleri

Hatayı tekrarlamak için takip edilmesi gereken adımlar

1. Pin eklerken pin içeriğine imaj ekle
2. Birkaç gün sonra kontrol et

Pano detay ekranı geliştirilmeli

Açıklama

• Panoya eklenen Pin'ler bu ekranda listelenmeli.

• Bu ekrandaki Pin'ler biraz küçük olmalı. Pinler ekranındaki Pin'ler gibi büyük Pin'ler kullanılırsa tek bakışta etkinlik hakkında fikir edinmek zor olur. Örneğin Karadeniz adlı Pano üzerinden Karadeniz gezisi yapmak istiyorum. Ekrana baktığımda kabaca hangi Pin'ler olduğunu görmeliliyim ki bana gezi hakkında fikir versin. Yoksa sürekli ekranı kaydırıp Pin'leri tek tek görmek gerekecek.

• Pin'ler sürüklenerek sıralanabilmeli.

Ek İçerik

• Aşağıdaki tasarım fikir verebilir. Video fotoğrafının yanındaki iki çizginin üzerine gelince cursor değişiyor ve tutan el gibi görünüyor. Burada click yapıp videolar sıralanabiliyor.

SEO için FriendlyId kullanılmalı

Açıklama

Google aramalarında öne çıkabilmek için FriendlyId kullanılmalı.

Hesap oluştururken gizlilik politikası ve kullanım koşullarının onaylanmış olacağına dair mesaj eklenmeli

Açıklama

Kayıtlanma ekranında (belki giriş ekranında da) kayıt ol butonunun altına şu tarz bir mesaj eklenmeli:
Hesap oluşturarak, Gizlilik Politikası, Kullanım Koşulları ve Çerez Politikası kabul etmiş sayılırsınız.

Pin index ekranında büyük veri olduğunda Pinler çok geç geliyor

Bu tasarım önerisi bir problemle mi ilgili? Lütfen detaylıca açıklayınız

Birden çok sorgu or ile birleştirildiğinden sorgu uzun sürüyor. Pluck ile büyük veri çekiliyor. Bunun yerine daha pagy ile pagination'a izin veren bir sorgu yazmak gerekiyor.

Sizin çözümünüz

Arel kullanılabilir. İncelemek lazım.

Counter cache bilgisini arada yenilemek gerekiyor

Açıklama

Herhangi bir sebeple counter cache için gerekli callback'ler çalışmazsa count bilgilerinde tutarsızlık olaşabilir. Bu durumu engellemek için belli aralıklarla bu bilgiyi yenilemek gerekiyor. Kullanılan counter_culture gem haftada bir counter_culture_fix_counts metodunun çalıştırılmasını öneriyor. Bunu bir background job ile çalıştırmak lazım. Production'da bu düzenleme yapılmalı. Gem'in konuyla ilgili dokümanı: https://github.com/magnusvk/counter_culture#manually-populating-counter-cache-values

Count bilgisi için counter cache kullan

Açıklama

Şu an sorun değil ama ilerde veri büyüdüğünde gerekli olacak. Bunun için rails'in çözümü olan counter_cache ya da daha gelişmiş özellikleri olan counter_culture kullanılabilir.

Ayarlar ekranları eklenmeli

Açıklama

Eklenmesi gereken ekranlar:

• Profili Düzenle
• Kullanıcı Adını Değiştir
• E-postayı Değiştir
• Parolayı Değiştir

Pin detay ekranında gösterilen oluşturulma saati tıklanabilir olmamalı

Bu tasarım önerisi bir problemle mi ilgili? Lütfen detaylıca açıklayınız

Pin detay ekranında Pin'in oluşturulma saatine tıklanabiliyor ama birşey olmuyor.

Sizin çözümünüz

Saat bilgisine tıklanamaması gerekiyor. Disabled edilebilir.

Ekran görüntüleri

En sondaki Pin için Panoya Ekle paneli doğru konumlanmıyor

Hatayı açıklayın

Küçük ekranlarda bir Pinlerin listelendiği bir sayfada son Pin içini Panoya Ekle paneli doğru çalışmıyor. Sayfa sürekli panelin üzerine doğru scroll yapıyor. Container'ın içindeki son eleman olduğu için, açılan panele yeterli alan kalmıyor sanırım. Pin'in altına gerekli boşluk bırakılırsa sorun olmuyor.

Ekran görüntüleri

panoya.ekle.hatasi.mp4

Hatayı tekrarlamak için takip edilmesi gereken adımlar

1. Pinler sayfasına git
2. Sayfanın en altına ilerle
3. Son Pin üzerindeki panoya ekle butonuna tıkla

Ekran küçüldüğünde imajlar en boy oranına göre küçülmüyor

Hatayı açıklayın

Ekran küçüldüğünde imajların da ekran boyutuna göre küçülmesi gerekiyor. Şu an imajların boyutu değişmiyor. Bu yüzden küçük ekranlarda yana doğru imaj taşması oluyor

Beklenen davranış

Ekranla beraber imajların da küçülmesi

Mevcut davranış

Ekranla beraber imajalr küçülmüyor

Ekran görüntüleri

Muhtemel çözüm

CSS ile çözülebilir

Hatayı tekrarlamak için takip edilmesi gereken adımlar

1. Pinlerin listelendiği ekrana git
2. Ekranı en küçük haline getir
3. Sağa doğru scroll yap

Pin üzerinde Pin'in eklendiği Panolar (Pin sahibinin) gösterilmeli

Açıklama

Pin üzerinde Pin'in eklendiği Panolar (Pin sahibinin) gösterilmeli. Bunun için component oluşturulabilir. Bir pano için aşağıdaki gibi, birden çok pano için x panosuna ve 4 tane daha panoya (link olacak şekilde) eklendi şeklinde gösterilebilir. Linke tıklandığında da pin'in eklendiği bütün panolar açılan modalda gösterilebilir.

Konum adı ve/veya ilçe-şehir-ülke bilgisi Pin üzerinde gösterilmeli

Bu tasarım önerisi bir problemle mi ilgili? Lütfen detaylıca açıklayınız

Konum bilgisi hiçbir yerde gösterilmiyor.

Sizin çözümünüz

Konum adı Pin üzerinde gösterilmeli. Aynı şekilde konum ilçe-şehir-ülke bilgisi de Pin üzerinde gösterilebilir.

Ekran görüntüleri

Uygulamadan gönderilen mailler için kullanınal mail adresi güncellenmeli

Açıklama

Şu an gmailden alınan mail kullanılıyor. Bunun yerine [email protected] kullanılmalı

Pin kapak fotoğrafı yükseklik/en oranı çok yüksek olmamalı

Bu tasarım önerisi bir problemle mi ilgili? Lütfen detaylıca açıklayınız

Bu oran yüksek olduğunda Pinlerin listelendiği ekranda tek pin ekranı kapatabiliyor.

Sizin çözümünüz

Kapak fotoğrafı eklerken sınırlama getirilebilir. Ya da gösterirken scale edilebilir.

Ekran görüntüleri

N/A

Panoya ekle paneli üzerinden pano oluşturulabilmeli

Açıklama

Turbo ile bu kolayca yapılabilir. Youtube'da buna benzer bir özellik var. Ekran görselleri:

Henüz verisi olmayan ekranlar için bilgi verici mesajlar eklenmeli

Açıklama

Örneğin yeni kayıt olan biri için Pinler ekranı boş görünecek. Oraya aşağıdaki tarzda birşeyler eklenebilir:

Locale (Dil) bilgisi veritabanında tutulmalı

Bu tasarım önerisi bir problemle mi ilgili? Lütfen detaylıca açıklayınız

Locale bilgisi şu an cookie'de tutuluyor. Kişi başka başka bir tarayıcıda oturum açtığında cookie bilgisi olmadığı için dil varsayılana dönüyor.

Sizin çözümünüz

Bunun yerine dil bilgisi veritabanında tutulursa, farklı bir tarayıcıdan oturum açıldığında dil bilgisi doğru gelecektir. Ayrıca varsayılan dil bilgisi gelen requet'in konumuna göre belirlenmeli. Yani istek türkiyeden geliyorsa varsayılan locale tr olmalı.
Ayrıca link paylaşımlarında url içinde locale olduğu için link'i açan kişinin tarayıcısında site üzerinde dil değişikliğine sebep olabilir.

Güncelleme:
Rails guide'ına göre locale bilgisini request header'ından almak konumdan almaya göre daha iyi. Oturum açanlar için de locale bilgisini db'de tutmak mantıklı. Ayrıca locale bilgisini cookie'de tutmak kesinlikle önerilmiyor. Kullanıcı, linki paylaşırken ne görüyorsa, linki açan da onu görmeli.

Ekran görüntüleri

N/A

Pano birleştirme özelliği eklenmeli

Açıklama

Pano birleştirme özelliği eklenmeli. Bir panodaki pinler başka bir panoya taşınabilmeli. Taşımadan sonra ilgili pano silinmeli.

Production için ssl'i aktif et

Açıklama

config/environments/production.rbde force_ssl: true olmalı.

SEO ayarı yapılmalı

Açıklama

SEO ayarı için şu gem kullanılabilir: meta-tags

Ekranlara minimum yükseklik eklenmeli

Bu tasarım önerisi bir problemle mi ilgili? Lütfen detaylıca açıklayınız

Veri olmayan ekranlarda footer yukarı kayıyor.

Sizin çözümünüz

Bunu engellemek için minimum yükseklik eklemek gerekiyor.

Ekran görüntüleri

Pin ile ilgili css kodları javascript/styles/controllers klasörü altına taşınmalı

Açıklama

Pin ile ilgili css kodları javascript/styles/theme/custom.scss dosyasında tutuluyor. Bu kodları javascript/styles/controllers klasörüne taşımak gerekiyor.

MoreButtonComponent genel kullanım için güncellenmeli

Bu tasarım önerisi bir problemle mi ilgili? Lütfen detaylıca açıklayınız

Komponent şu an genel kullanıma uygun değil. Her model için yeniden oluşturmak gerekiyor. Bunun yerine genel bir komponent olmalı ve dinamik bir şekilde içine link eklenebilmeli.

Sizin çözümünüz

Block ile linkler bu komponent'e eklenebilir.

Ekran görüntüleri

Ek kaynaklar

Örnek: https://primer.style/view-components/components/dropdown

Panoya ekle butonun eklendi-eklenmedi görseli doğru çalışmıyor

Hatayı açıklayın

Bir Pin pano değil de bölüm'e eklenmişse butonun görüntüsü yanlış oluyor. Added class'ı olması gerekirken added class'ı eklenmiyor butona.

Beklenen davranış

Butonun rengi eklendi (siyah) olmalı

Pano arama ekranındaki pano adları daha görünür olmalı

Açıklama

Pano arama ekranındaki pano adları soluk görünüyor. Panolar ekranındaki gibi koyu olması daha iyi olur.

Ekran görüntüsü

Uygulama içinde Email yerine Username kullanılmalı

Bu tasarım önerisi bir problemle mi ilgili? Lütfen detaylıca açıklayınız

Uygulama içinde Email yerine Username kullanmak uygun olur. Email yerine Username ile de giriş yapılabilmeli.

Sizin çözümünüz

Kayıtlanma aşamasında Username belirlenebilir. Diğer uygulamalar (Google, Facebook, Twitter) üzerinden kayıt (giriş) yapıldığında kullanıcı adı servisten gelen isme göre random atanacak. Sonradan kullanıcı isterse kullanıcı adını güncelleyebilecek. Örneğin Google'dan gelen isim 'Emre Demir' olsun. Bu durumda emredemir1 ya da emredemir2 gibi kullanıcı adları otomatik oluşturulacak.

Ekran görüntüleri

N/A

Pin detay ekranındaki harita çalışmıyor

Açıklama

Pin detay ekranındaki harita çalışmıyor.

Board (Pano) modeli ve ekranları eklenmeli

Açıklama

Pinleri gruplamak için Pano modeli gerekli. Bu sayede Pin'ler derli toplu olabilecek. Paylaşımlar da daha anlamlı olacak.

Küçük ekranlar için nav menü düzenlenmeli

Bu tasarım önerisi bir problemle mi ilgili? Lütfen detaylıca açıklayınız

Küçük ekranlarda nav menüdeki bağlantılar direk gösterilmeli. Örneğin şu an küçük ekrandayken 'Panolar' bağlantısını görmek için hamburger butonuna tıkladıktan sonra bir de kullanıcı fotoğrafına tıklamak gerekiyor. Bunun yerine 'Panolar' bağlantısı en üst seviyede olmalı. Yani hamburger butonuna tıklandığında direk görünmeli.

Sizin çözümünüz

Html/Css düzenlemesi yapılmalı

Ekran görüntüleri

Mobilde Pin düzenle ekranında kapak fotoğrafı ekrandan taşıyor

Hatayı açıklayın

Mobilde Pin düzenle ekranında kapak fotoğrafı ekrandan taşıyor

Beklenen davranış

Ekrana sığrılması gerekiyor

Mevcut davranış

Sağa doğru kayma oluyor

Ekran görüntüleri

Muhtemel çözüm

Pin formunda imaja responsive özelliği vermek yeterli olabilir

Hatayı tekrarlamak için takip edilmesi gereken adımlar

Kapak fotoğrafı ile beraber Pin oluştur
Mobildeyken Pin düzenleme ekranına gir