Skip to content
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

Genel düzeltmeler yapıldı #248

Merged
merged 7 commits into from
Dec 23, 2024
Merged

Conversation

GokturkTalha
Copy link
Member

@GokturkTalha GokturkTalha commented Dec 23, 2024

Yapılan Değişikler

Lütfen yaptığınız değişikleri önizleme olması açısından kısaca açıklayınız. Yaptığınız değişiklikler bir hatayı düzeltiyorsa veya bir özellik talebini çözüyorsa, bu soruna bağlantı verdiğinizden emin olun.

Değişiklik Türü

Katkınız proje üzerinde ne tür bir değişikliğe eşit geliyor?
Uygun olan kutulara x işareti koyun.

  • Dökümantasyon eklemeleri yapıldı (yeni sayfalar, yeni uygulamalar)
  • Dökümantasyon düzeltmeleri yapıldı (sayfa iyileştirmeleri, yazım hatalarının giderilmesi)
  • Koda katkı yapıldı (CSS/JS ya da diğer unsurlar)
  • Hatalar giderildi (kod ve diğerleri)

Kontrol Listesi

Uygun olan kutulara x işareti koyun. Bunları PR oluşturduktan sonra da doldurabilirsiniz. Bunlardan herhangi biri hakkında emin değilseniz, sormaktan çekinmeyin. Yardım etmek için buradayız! Bu sadece kodunuzu birleştirmeden önce nelere bakacağımıza dair bir hatırlatmadır.

  • Yapılan değişiklerin kontrolleri yerel sitede yapıldı.
  • Eklemelerim markdown ve yazım kurallarına uygun.
  • Sayfaya tablo ekledim/ tabloyu düzenledim. (tabloda gereksiz sütunlar yok)
  • Sayfa düzeni, diğer sayfalardan farklı değil.
  • Eklediğim uygulama(lar) açık kaynak.
  • Uygulama logosunu ekledim.
  • Eklediğim logo transparan.
  • Eklediğim tarayıcı eklentisi mahremiyet ve güvenlik anlamında tarayıcıya artılar ekliyor.
  • Eklediğim uygulamanın açık kaynak deposunu ve gizlilik politikalarını ekledim.
  • CSS/JS ve diğer kod düzeltmelerimde bir hata yok.
  • Giderilen hataları test ettim.

Diğer yorumlar

Bazı logo düzeltmeleri yapıldı, ana sayfada önerilenlere bir tane de mesajlaşma uygulaması olması adına Session eklendi.

Copy link
Member

@wiseweb-works wiseweb-works left a comment

Choose a reason for hiding this comment

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

Virgül kullanımıyla ilgili göze çarpan bir durum var. Tek bir gönderide farkedilmez muhtemelen ama içerik tüketmeye devam edildiği sürece tekrar eden bu örüngü/pattern farkedilecektir. İngilizcenin aksine bizde ya tekrar eden nesneleri ayırabiliriz ya da Özneden sonra belirli amaçlarla koyabiliriz. Fakat burada farklı amaçlar için de kullanılmış.

Kod yapısı veya tasarım açısından sorun yaratan bir durum yok. 👍🏻

docs/sosyal-aglar.md Outdated Show resolved Hide resolved
docs/mesajlasma-uygulamalari.md Show resolved Hide resolved
@GokturkTalha
Copy link
Member Author

Bazı gereksiz yerleride çıkarttım, kullanıcıya ekstra bilgi vermiyordu yada önemsiz bilgiler içeriyordu.

@wiseweb-works
Copy link
Member

Şu an finalize edilmiş hali mi?

@GokturkTalha
Copy link
Member Author

Şu an finalize edilmiş hali mi?

Evet başka bir sorun göremedim

docs/posta-maskeleme.md Outdated Show resolved Hide resolved
docs/sosyal-aglar.md Outdated Show resolved Hide resolved
docs/sosyal-aglar.md Outdated Show resolved Hide resolved
docs/sosyal-aglar.md Outdated Show resolved Hide resolved
Copy link
Member

@wiseweb-works wiseweb-works left a comment

Choose a reason for hiding this comment

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

Tamamdır ellerine sağlık 👍🏻

@GokturkTalha GokturkTalha merged commit 3474951 into GuvendeKal:main Dec 23, 2024
1 check passed
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.

2 participants