golang-tr / tour

Türkçe Go Turu
https://go-tour-turkish.appspot.com
BSD 3-Clause "New" or "Revised" License
18 stars 1 forks source link

Kod deneme bölümü çalışmıyor. #2

Closed kahredici closed 8 years ago

kahredici commented 8 years ago

İngilizcesinde sıkıntı yok ancak Türkçesinde "Sunucu bekleniyor..." ve sonuç ekrana gelmiyor. Sanırım 2 gün öncesine kadar sıkıntı yoktu.

roktas commented 8 years ago

Raporladığınız için teşekkürler. Bir güncelleme yapmamız lazım. An itibarıyla çevirilerde kritik olan content dizininde son değişiklikten bugüne neler olmuş?

$ git log --oneline --since 2014-09-29 -- content | wc -l
80

Toplam 80 değişiklik, epey hallice! Önce bunları bir güncelleyelim, sonrasında servisi tekrar ayağa kaldırırız.

roktas commented 8 years ago

Same issue: golang/go#15693 (the mentioned workaround has been applied).

adg commented 8 years ago

So you updated the tools repo before re-deploying? And it didn't work?

roktas commented 8 years ago

Yep ( "go get -u golang.org/x/tools/playground && goapp serve deploy .")

adg commented 8 years ago

Do you mean goapp serve or goapp deploy ? The former will not work, the latter will.

roktas commented 8 years ago

Sorry, there is a typo in my message. I mean the latter goapp deploy. Deployment seems successful.

adg commented 8 years ago

Can you show me the output of cd $GOPATH/src/golang.org/x/tools && git log -n 1 ?

roktas commented 8 years ago
$ cd $GOPATH/src/golang.org/x/tools && git log -n 1
commit c86fe5956d4575f29850535871a97abbd403a145
Author: Cezar Sa Espinola <cezarsa@gmail.com>
Date:   Wed Apr 13 18:57:13 2016 -0300

    imports: try importing pkg path from vendor when looking for pkg name

    Since CL 17728 goimports was using the file path to try finding packages
    inside its /vendor directory. This CL builds upon this previous work to
    also consider packages inside /vendor when converting an import path to
    a package name.

    Change-Id: I173472ac51e3f681d758ec52add60ffc04c62c1a
    Reviewed-on: https://go-review.googlesource.com/22020
    Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
    Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
roktas commented 8 years ago

I think I found my problem. Our repository is badly outdated, as you can see at https://github.com/golang-tr/go-tour-tr/blob/master/gotour/appengine.go#L18 (still using the old code from code.google.com which is not affected from a go get -u golang.org/x/tools/playground) . I'll try to update the whole repository.

adg commented 8 years ago

@roktas that would make sense. Let me know if you need advice.

roktas commented 8 years ago

@adg FYI, I've performed a full update from the upstream repository, and completed the Turkish translations. Everything seems ok at the moment as you can see at https://go-tour-turkish.appspot.com (ÇalıştırRun button). Thanks for your help.

roktas commented 8 years ago

/cc @fatih

Fatihçiğim bizim öğrenciler kayboldu, ben el atmak zorunda kaldım. Hızlı bir güncelleme yaptım, pek çok sorun olduğuna eminim. Bebişi sallarken arada bulabileceğin ölü vakitlerde şöyle hızlıca bir göz atarsan makbule geçer. ;-) Özellikle "tür onayları", "metodlar" ve "arayüzler". Gerekli gördüğün düzenlemeleri çabucak Web arayüzü üzerinden yapabilirsin.

P.S. Servisin kanonik adresi olan https://tur.golang.org.tr henüz çalışmıyor, bilahare bakacağım ona da.

roktas commented 8 years ago

@kahredici Tur'u güncelledim. Bahsettiğiniz sorun da çözülmüş görünüyor, dolayısıyla raporu kapatıyorum. Sorun tespit ederseniz çekinmeden raporlayın lütfen.

roktas commented 8 years ago

@fatih İpucu: Türkçe çeviriyi orphaned bir dal üzerine bindirdiğimden (an itibarıyla) son gönderide İngilizce/Türkçe yan yana görebiliyorsun (çevirinin pek çoğu eski sürümden geliyor).

roktas commented 8 years ago

@adg We are giving up using a custom domain for Turkish Go Tour[1]. Could you please change the link to https://go-tour-turkish.appspot.com/? I can create an issue if you prefer.

[1] An SSL certificate required and it must be renewed periodically. Too much burden for us.