github community added credits and footer #61
Conversation
|
Merhaba @asiminnesli degisiklik icin tesekkur ederiz, Evrim Agaci Backend kismini bilmiyorum belki oradan veriyi kolayca girebiliyorlardir. Simdi JSON'i duzenlemekle ugrasmak gerekecek, bilmiyorum @cagrimertbakirci hangisini tercih edersiniz? Suan Footer'dan GitHub'a tiklayinca bos bir sayfa gelmesi hos durmayacak, o yuzden bu PR'i #52 daki onerinizle beraber implement edene kadar bekletebilir miyiz? Ya da bu PR'i sadece Kunye ilgili bir degisiklik olarak commitlemek daha dogru. |
| try { | ||
| const result = await axios.get(`https://evrimagaci.org/api/v2/periodum/credits`) | ||
| res.status(200).send(result.data) | ||
| const result = require(`../credits.json`) |
There was a problem hiding this comment.
Bunu global yapabiliriz. Surekli degismeyecegi icin, degisecegi zaman da zaten deployment gerekecek. Ama GitHub API'nina baglarsak bu scope'da kalir.
|
@asiminnesli bu konuyla ilgili bisey yapacak misiniz? |
credits in json ile taşınması konusunda bir fikir birliği oluşmadı sanırım ondan dolayı bu PR ı geri çekiyorum . |
|
Merhaba @asiminnesli PR'i geri cekmenize gerek yoktu. Evet o kisimda ufak bir eksigimiz olmus, kusura bakmayin. Aspect ratio olayi surada cozuldu #39 Sizin degisikliginiz sadece Kunye ile ilgili (credits.json) olacak sekilde yeniden bir PR olusturmaniz mumkun mu? GitHub olmasin footer'da, bos sayfa koymayalim lutfen. |
Açıklama
Künye kısmı evrimagaci.com üzerinde api ile alınıyordu onun yerine proje kendi içerisinde bir json dosyasında tutulacak şekilde düzenlendi.
Projesinin künye ve footer alanlarına Github Komünitesinin logosu eklendi.
Alakalı Hata (Issue): #52 (GitHub Komünitesi künyeye ve footer'a eklenmeli. )
Değişiklik Tipi
Ekler
Nasıl Test Edilir?