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

Diğer ödeme yöntemleri kaldırıldı #256

Merged
merged 1 commit into from
Dec 31, 2024
Merged

Conversation

GokturkTalha
Copy link
Member

@GokturkTalha GokturkTalha commented Dec 31, 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

Dijital anlamda güvenlik sağlamadığından ve biraz paranoyaklık olarak hisettiğimden ilgili sayfanın kaldırılması belki daha doğrudur. Bu PR altında tartışabiliriz.

@wiseweb-works
Copy link
Member

Sonrasında tekrar ekleyecek miyiz yoksa bu kısımdan vaz mı geçtik?

@GokturkTalha
Copy link
Member Author

Sonrasında tekrar ekleyecek miyiz yoksa bu kısımdan vaz mı geçtik?

Dijital anlamda güvenlik sağlamadığından ve biraz paranoyaklık olarak hisettiğimden ilgili sayfanın kaldırılması belki daha doğrudur. Bu PR altında tartışabiliriz.

@wiseweb-works
Copy link
Member

İçerik olarak bir yorumum ya da bakış açım yok maalesef. Daha önce kullanmadım ya da deneyimim olmadı maalesef. O yüzden bir şey diyemeyeceğim.

Fakat kod ve dosya yapısı açısından işlem doğru yapılmış. Başkaca bir sayfada kalan bir atıf yok ise tamamen kaldırılmış olacak. 👍🏻

@GokturkTalha GokturkTalha merged commit a7ca716 into main Dec 31, 2024
2 checks passed
@GokturkTalha GokturkTalha deleted the diger-odeme-yontemleri branch December 31, 2024 19:27
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