Skip to content

Conversation

samilboyalikli
Copy link
Contributor

  • Sıfırla butonu eklendi.
  • Bu buton hem lokali hem harita renklerini hem de counterı sıfırlıyor.
  • Bölgeler butonu kaldırıldı.

samilboyalikli and others added 17 commits May 2, 2024 23:34
explanations of some codes
some explanations about some d3 methods
some description about some codes
some description about some codes
added bases with counters
added codes for counters place in localstorage
added bases codes
last changes
- a versiyon for main branch
Copy link
Owner

@ozanyerli ozanyerli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Elinize sağlık.

Bence sıfırla butonu indirme butonun üstünde olsa daha güzel olur. Bir de Github logosunun kaydığını görüyorum. Orijinal yerinde kalmalı.

image

Bu değişikliklerden sonra mergeleyebiliriz.

@ozanyerli ozanyerli linked an issue May 26, 2024 that may be closed by this pull request
@samilboyalikli
Copy link
Contributor Author

Bu yolladığım kodlarda dediğiniz şekilde biçimlendirmeye çalıştım: #11

@ozanyerli ozanyerli mentioned this pull request May 27, 2024
@samilboyalikli
Copy link
Contributor Author

Git Bash'ten PR'e ulaşıp logo ve buton konumlarını değiştirebildim ama geri pushlayıp PR'i güncelleyemedim. Bu yüzden Github'tan direkt manuel olarak istediğiniz değişiklikleri yapmaya çalıştım. Umarım atladığım bir nokta olmamıştır.

@ozanyerli ozanyerli self-requested a review June 2, 2024 10:18
Copy link
Owner

@ozanyerli ozanyerli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Elinize sağlık. Bu hali uygundur. Mergeliyorum.

PR değil de branch üzerinde çalışmanız gerekiyor, muhtemelen o sebeple sorun yaşadınız. PR'ı GitHub'ın eklediği bir fonksiyonalite olarak düşünebilirsiniz, normalde git'in bir parçası değil. Aslolan şey sizin açtığınız branch. O sebeple git bash değil direkt git komutlarıylaresetButton-24-05-24 branchine geçip orada değişiklikleri yapabilirsiniz.

@ozanyerli ozanyerli merged commit 6961b5e into ozanyerli:master Jun 2, 2024
@samilboyalikli
Copy link
Contributor Author

Bunu öğrendiğim çok iyi oldu teşekkürler. Git Bash'te elli saat PR açmaya çalıştım :) Branch üzerinden gitmek gerekiyormuş cidden.

@ertugrul-yazilim
Copy link

Peki türkiyenin ilçelerinin de yer aldığı bir arayüz oluşturamaz mısın. iller ve ilçeler bir arada. bir türlü yapamadım

@samilboyalikli
Copy link
Contributor Author

Peki türkiyenin ilçelerinin de yer aldığı bir arayüz oluşturamaz mısın. iller ve ilçeler bir arada. bir türlü yapamadım

Selamlar.

Öncelikle ben contributor'ım. Projenin sahibi sevgili Ozan Yerli (https://github.com/ozanyerli). Ben şehirlerin sayısını hem arayüzden hem de local storage'den sıfırlayan kısmı ekledim.
Codebase'e epeydir bakmamıştım şu anda tekrar göz attım. Anladığım kadarıyla tr-cities.json dosyasında ilçelerin koordinatlarını ve isim/numaralarını tıpkı Ozan'ın yaptığı gibi eklemeniz gerekiyor. Sonrasında turkeyvisited.js'i ona göre revize etmek gerekebilir.

Yalnızca şöyle bir durum var: İlçelerin eklenmesi beklediğiniz etkiyi oluşturmayabilir ve feature creep yaparak uygulama trafiğini olumsuz yönde etkileyebilir. Ben şahsen gittiğim illerde hangi ilçeleri ziyaret ettiğime pek dikkat etmiyorum, aynı şekilde etrafımdaki insanların da dikkat ettiğini görmüyorum. Elbette bu benim tecrübem, yanılıyor olabilirim.

@ertugrul-yazilim
Copy link

ertugrul-yazilim commented Aug 28, 2025 via email

@samilboyalikli
Copy link
Contributor Author

samilboyalikli commented Aug 28, 2025 via email

@ertugrul-yazilim
Copy link

ertugrul-yazilim commented Aug 28, 2025 via email

@samilboyalikli
Copy link
Contributor Author

samilboyalikli commented Aug 28, 2025 via email

@ertugrul-yazilim
Copy link

ertugrul-yazilim commented Aug 28, 2025 via email

@samilboyalikli
Copy link
Contributor Author

samilboyalikli commented Aug 28, 2025 via email

@MetaMsa
Copy link

MetaMsa commented Sep 2, 2025

Harita GeoJson standardıyla yapılıyor. İnternetten araştırabilirsiniz bunun için harita üzerinden otomatik GeoJson çıktısı veren siteler var. Ancak her ilçe için GeoJson çıktısı almak uzun sürebilir.

Siz anlatınca kolay da bi de bana sorun 28 Ağu 2025 Per 23:07 tarihinde Samil BOYALIKLI @.> şunu yazdı:

samilboyalikli left a comment (ozanyerli/turkeyvisited#10) <#10 (comment)> Anladım. Öncelikle sıfırdan harita kurgulamak zorunda değilsiniz. Mevcut datasete ilçeler eklenerek ilerlenebilir. PR'lara bakarsanız bir arkadaşımız KKTC'ni de eklediği bir versiyon yollamış mesela ( #13). Buradaki kodları incelerseniz, aslında contributor adayı yalnızca tr-cities.json dosyasına KKTC'nin koordinatlarını ve özelliklerini ekleyip turkeyvisited.js'i buna göre revize etmiş. Ben ne yazık ki yoğunluğumdan dolayı buna vakit ayıramam ama burada kilit noktanın tr-cities.json dosyası olduğunu düşünüyorum. Buradan yola çıkarsanız istediğiniz sonuca daha rahat ulaşabilirsiniz. Ertugrul K.Sahin @.
>, 28 Ağu 2025 Per, 22:53 tarihinde şunu yazdı: > ertugrul-yazilim left a comment (ozanyerli/turkeyvisited#10) > < #10 (comment)> > Aslında ben yazılıma sadece ilgiliyim dillere hakim değilim. Cursor ile > fikrimi hayata geçirmeye çalıştım o da beceremedi sizin gibi dinamik > harita > eklemeyi > > 28 Ağu 2025 Per 22:51 tarihinde Samil BOYALIKLI @.> > şunu yazdı: > > > samilboyalikli left a comment (ozanyerli/turkeyvisited#10) > > < > #10 (comment)> > > > Anlıyorum. Gayet güzel düşünmüşsünüz, özellikle memur adaylarının > > kullanabileceği birşey bu. Peki hangi noktada problem oldu? > > > > Ertugrul K.Sahin @.>, 28 Ağu 2025 Per, 22:41 > > tarihinde şunu yazdı: > > > > > ertugrul-yazilim left a comment (ozanyerli/turkeyvisited#10) > > > < > > > #10 (comment)> > > > > > > Aslında konu ilçe eklerken başka bir amaçtı. Tercih ekranı yapmak > > > istiyorum > > > kişiler türkiyedeki il ve ilçeleri seçerek hangi atama noktasına > tercih > > > verdiğini türliye haritası üzerinden görsün istedim > > > > > > 28 Ağu 2025 Per 22:32 tarihinde Samil BOYALIKLI @.> > > > şunu yazdı: > > > > > > > samilboyalikli left a comment (ozanyerli/turkeyvisited#10) > > > > < > > > > > > #10 (comment)> > > > > > > > > > > > > > > Peki türkiyenin ilçelerinin de yer aldığı bir arayüz oluşturamaz > > mısın. > > > > iller ve ilçeler bir arada. bir türlü yapamadım > > > > > > > > Selamlar. > > > > > > > > Öncelikle ben contributor'ım. Projenin sahibi sevgili Ozan Yerli ( > > > > https://github.com/ozanyerli). Ben şehirlerin sayısını hem > arayüzden > > > hem > > > > de local storage'den sıfırlayan kısmı ekledim. > > > > Codebase'e epeydir bakmamıştım şu anda tekrar göz attım. Anladığım > > > > kadarıyla tr-cities.json dosyasında ilçelerin koordinatlarını ve > > > > isim/numaralarını tıpkı Ozan'ın yaptığı gibi eklemeniz gerekiyor. > > > > Sonrasında turkeyvisited.js'i ona göre revize etmek gerekebilir. > > > > > > > > Yalnızca şöyle bir durum var: İlçelerin eklenmesi beklediğiniz > etkiyi > > > > oluşturmayabilir ve feature creep yaparak uygulama trafiğini olumsuz > > > yönde > > > > etkileyebilir. Ben şahsen gittiğim illerde hangi ilçeleri ziyaret > > > ettiğime > > > > pek dikkat etmiyorum, aynı şekilde etrafımdaki insanların da dikkat > > > > ettiğini görmüyorum. Elbette bu benim tecrübem, yanılıyor > olabilirim. > > > > > > > > — > > > > Reply to this email directly, view it on GitHub > > > > < > > > > > > #10 (comment)>, > > > > > > > > > > or unsubscribe > > > > < > > > > > > https://github.com/notifications/unsubscribe-auth/BUOIQJBTRFK6SIQQHIBVYVL3P5KOFAVCNFSM6AAAAACFCJH76GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTEMZUG4YDIMBRGY> > > > > > > > > > > . > > > > You are receiving this because you commented.Message ID: > > > > @.> > > > > > > > > > > — > > > Reply to this email directly, view it on GitHub > > > < > > > #10 (comment)>, > > > > > > or unsubscribe > > > < > > > https://github.com/notifications/unsubscribe-auth/BBN63F6XGZOILJF5HOZ2GI33P5LO5AVCNFSM6AAAAACFCJH76GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTEMZUG4ZDMMZTGI> > > > > > > . > > > You are receiving this because you authored the thread.Message ID: > > > @.> > > > > > > > — > > Reply to this email directly, view it on GitHub > > < > #10 (comment)>, > > > or unsubscribe > > < > https://github.com/notifications/unsubscribe-auth/BUOIQJBHWL4QOZC6CDT6J6L3P5MT5AVCNFSM6AAAAACFCJH76GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTEMZUG42TCMBRG4> > > > . > > You are receiving this because you commented.Message ID: > > @.> > > > > — > Reply to this email directly, view it on GitHub > < #10 (comment)>, > or unsubscribe > < https://github.com/notifications/unsubscribe-auth/BBN63F6HUFFFKOMDZFIEGLD3P5M4ZAVCNFSM6AAAAACFCJH76GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTEMZUG42TMNBTGU> > . > You are receiving this because you authored the thread.Message ID: > @.> > — Reply to this email directly, view it on GitHub <#10 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/BUOIQJEVWSJJUIRU3CLGA733P5OPJAVCNFSM6AAAAACFCJH76GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTEMZUG44TANRXHA . You are receiving this because you commented.Message ID: @.>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Seçili olan illeri temizleme butonu
4 participants