Closed ogirginc closed 2 years ago
@ogirginc bunu merge edelim mi hazirsa?
Discourse logo sorunu var bu PR'da. Düzeltmemiştim en son. Sonra öyle kaldı. İstersen bi dene şansını 😊
@ogirginc bunu merge edelim mi hazirsa?
Discourse logo sorunu var bu PR'da. Düzeltmemiştim en son. Sonra öyle kaldı. İstersen bi dene şansını 😊
Şimdi baktım da bir sorun yok görünüyor. Başka birşey mi var?
https://user-images.githubusercontent.com/447588/173178950-65b17334-0bf6-4738-91da-15b6ab10f334.mov
@ogirginc hee anladım. sorry. sen fa-icon dion :)
bizim kullandığımız versiyon fontawesome 4 kullanıyor. https://fontawesome.com/v5/icons/discourse?s=brands discourse 5'de gelmiş. bunu kullanmak lazım gem olarak https://github.com/tomkra/font_awesome5_rails ama bence çok önemli değil. iconsuz dursun şimdilik ya da sallama bişi koyabilirim ben istersen. fontawesome upgrade'i için issue açarım. ne dersin? @ogirginc
Bana OK. Merge'le istersen direkt 👍
@ogirginc bunu merge edelim mi hazirsa?