Genel düzenlemeler #244
Genel düzenlemeler #244
Conversation
GokturkTalha
requested review from
relliv,
islemci,
wiseweb-works,
conmecha and
odest
|
Sanırım kendi PR'larımı çekmemem proje güvenliği açısından daha iyi olacaktır. Maintainer arkadaşlar arasından inceleyen birisi PR'ı olumlu bulursa çekebilir. |
There was a problem hiding this comment.
1- Görseller ile ilgili imgbot adlı aracı kullanmamız iyi olabilir. Görsellerin çözünürlüğü ve boyutunu dikkate almadan yüklüyoruz sonrasında o gerekli aralıklarla gelip görselleri tarıyor ve sıkıştırma/resize etme imkanı varsa pull request atıyor.
2- Markdown'ın içine çok fazla miktarda html yazıyoruz. Bu da pek mantıklı bir kullanım olmuyor. Özellikle docsify özelinde gerekli olan bir kısım yok ise mümkün olduğunda markdown özellikleri ile yapılması güzel olur diye düşünüyorum.
3- Bazı kısımlarda önceki PR'lar ile eklenmiş linkler kaldırılmış. Bunun bir sebebi var mı acaba?
| <div class="title"> | ||
| <a href="https://guvendekal.org/#/sifre-yoneticileri?id=-vaultwarden" target="_blank" style="color: inherit; text-decoration: none;">Vaultwarden</a> |
There was a problem hiding this comment.
Vaultwarden neden kaldırıldı? Acaba satır eklenmek istenirken yanlışıkla mı silindi?
There was a problem hiding this comment.
Vaultwarden, sunucu kurmaya yaradığından kaldırıldı. İleride self-host rehberi açarsak oraya ekleyebiliriz ancak Keepass, Bitwarden ve Proton Pass'in yanında farklı kategoride bir uygulama olarak kalıyordu. Örneğin Vaultwarden ile kurduğun sunucuya Bitwarden ile bağlanıyorsun yada başka istemcilerle.
There was a problem hiding this comment.
Vaultwarden bir Bitwarden forku değil miydi? 🤔 ben mi yanlış hatırlıyorum. Hatta kendi sunucumda da var. Basit bir docker container'ına kurup bitwarden ile bağlayabiliyoruz. Sunucu kurmaktan kastınız nedir tam olarak?
There was a problem hiding this comment.
Benim hatırladığım kadarıyla Bitwarden forku ancak Rust ile tekrar yazılmıştı sanırım. Sonuç olarak bir istemci değil bildiğim kadarıyla şifrelerin depolanmasına yarayan bir sunucu işlevi gördüğünü hatırlıyorum. Ben eklemiştim Vaultwarden'i ancak ne işe yaradığını tam hatırlamıyorum.
There was a problem hiding this comment.
Tamam şimdi anladım durumu. Sunucuya yüklediğimiz anlam ve görev farklı.
There was a problem hiding this comment.
Aynen aynen ben biraz yanlış anlattım sanırım. İşteyim ondan tam kafamı kullanamıyorum. Vaultwarden, Bitwarden sunucularını kullanmak istemeyen (kendi şifre sunucusunu kurmak isteyen) kullanıcılara 3. parti bir sunucu sağlıyor.
| @@ -20,10 +20,10 @@ Bu gibi sebeplerden ötürü, Chromium'dan sonra en çok kullanılan tarayıcı | |||
|
|
|||
| | Tarayıcı | Tabanı | Açık Kaynak | Lisans Türü | Platform Desteği | | |||
| | --- | :---: | :---: | :---: | :---: | | |||
| | <span style="display: inline-block; vertical-align: middle;"><img src="docs/images/firefox.png" alt="Firefox" style="width: 30px; height: 30px;"> </span> <span style="display: inline-block; vertical-align: middle;"> [**Firefox**](https://www.mozilla.org/en-US/firefox/new/) | Firefox | <span style="color: green;">✓</span> | MPL | <i class="fa-brands fa-windows"></i> <i class="fa-brands fa-apple"></i> <i class="fa-brands fa-linux"></i> <i class="fa-brands fa-android"></i> <i class="fa-brands fa-app-store-ios"></i> | | |||
There was a problem hiding this comment.
Kaldırılan linkler. Tek girdiyi seçtim ama birçok dosyada bu durum var.
There was a problem hiding this comment.
Bütün tablolarda olan linkleri kaldırdım, başlıklar altında kalan kısma linkleri girmemiz daha sağlıklı olur bence.
| @@ -41,16 +40,6 @@ Bulut tabanlı çalışan [**Bitwarden**](https://bitwarden.com/) açık kaynakl | |||
|
|
|||
| Bitwarden'da tüm veriler kullanıcı cihazında şifrelenir ve şifre çözümlenir; bu da herhangi bir veri ihlali yada çalınma durumlarında, hırsızların eline sadece şifrenin hash değerlinin geçeceği anlamına gelir. Bitwarden'in ücretli ve ücretsiz sürümleri bulunur. Bitwarden'in ücretli sürümü TOTP, gelişmiş 2FA seçenekleri, 1 GB şifreli dosya saklama gibi ek özellikler sunar. | |||
|
|
|||
| ### <span style="display: inline-block; vertical-align: middle;"><img src="docs/images/vaultwarden.png" alt="Vaultwarden" style="width: 50px; height: auto;"> </span> <span style="display: inline-block; vertical-align: middle;"> Vaultwarden | |||
There was a problem hiding this comment.
Vaultwarden'ın kaldırılması ile mi ilgili?
There was a problem hiding this comment.
Evet yukarıda belirtildi.
|
Markdown içine html yazmamızın nedeni daha estetik sayfalar oluşturabilmek için aslında. Markdownda böyle komplike tasarımlar olmuyor ne yazık ki. |
Görsel botunu kurabiliriz, eve geçince bakacağım. |
Bunu anlıyorum fakat her şey için de HTML kullanmamak lazım. Düz a etiketi için bile html kullanmışız mesela. Kaldı ki başka yerlerde bunun markdown versiyonuna zaten aşinayız. Özellikle bir yerde farklılık gerekirse mesela (renk, decoration vs) bunu zaten css ile sağlayabiliyoruz. |
Başlıkların yanında ki görsellerin belli bir boyuta (50px) indirmek için aynı zamanda yanında yazan yazıya oranlamak için kullanıyoruz. Bunu da Markdown ile yapamıyoruz. Başka aklıma gelmiyor farklı tarzda html kullandığımız . |
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
xişareti koyun.Kontrol Listesi
Uygun olan kutulara
xiş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._Diğer yorumlar
Tablolardaki bağlantılar kaldırıldı, sayfa içinde başlık altında bağlantı eklemeleri yapılması daha iyi olacaktır. Bazı transparan olmayan ve estetik durmayan logolar güncellendi. Anasayfa uygulamalar kısmına eksik olan uygulamalar eklendi ve bazıları çıkartıldı. Vaultwarden sunucu oluşturmaya yaradığından kaldırıldı. Tarayıcı Eklentileri sayfası Güvenlik kategorisine eklendi.