ale coś chyba nie działa:
np.:
link: http://webpraca.net/modszy-programista-ruby
wyświetla: We’re sorry, but something went wrong. We’ve been notified about this issue and we’ll take a look at it shortly.
Przejrzyście się prezentuje. Wygodna nawigacja.
Znalazłem litrówkę a adresie url: http://webpraca.net/oferty/najnowsze/zarzdzanie
Chyba zamiast “zarzdzanie” powinno byc “zarzadzanie”.
Wiem że funkcjonalnie to nie robi różnicy ale delikatnie razi.
[quote=macbury]Kod serwisu jest udostępniany na licencji GPL na GitHubie: http://github.com/macbury/webpraca[/quote]
Yay, super. Mam nadzieję że chętnie usłyszysz kilka sugestii co do kodu?
formatowanie kodu się rozjeżdża, tabulatory i spacje nie idą w parze
kontrolery wcale nie muszą być takie duże. JobsController::home spokojnie może mieć 2 linijki, funkcjonalność może być w modelu i to łatwiej przetestować
testy… chociaż dla modeli!
ogólnie fajnie i fajnie że udostępniłeś na githubie
Zjazd jest, człowiek się nudzi, jakiegoś cucumbera z capybarą mu się walnąć zdarzy ;]
Jest jakiś jeden podstawowy scenariusz. Stack: cucumber, capybara, factory_girl. Jak będzie mi się na jakimś wykładzie nudzić to może nawet rspeca z kilkoma podstawowymi testami wrzucę ;]
macbury:
Definitywnie musisz dodać do README info o wrzucaniu tych wszystkich plików yml (jak spojrzysz w moje commity, to dodałem jakieś przykładowe i komunikaty błedów, które podpowiadają co zrobić). Najlepiej w ogóle jakbyś zrobił taska, który sam przerzuca examples w odpowiednie miejsca. Spróbuj sclonować sobie na czysto projekt i odpalić - zobaczysz, które miejsca trzeba poprawić. Jak człowiek się wywali na jakichś głupich yamlach (których jest tam łącznie 4 i nigdzie nie było informacji co musi być w środku, więc trzeba obczajać z w kodzie), to nie pośle Ci żadnego fajnego patcha
@gregorz to wina pluginu has_permalink(nie obsługuje on polskich znaków)
Testy: zabiorę się za nie dzisiaj(przy okazji dzięki drogus za przykładowe testy). Normalnie mam problem z testami jeżeli odpalam je na railsach chodzących na mysql(coś jest nie tak z gemem mysql i testy ładują się straaaaaaaaaaasznie długo). @hubertlepicki. hmm… 1.) to pewnie przez to że kawałki kodu kopiowałem z README @drogus: Pliki konfiguracyjne… Wiedziałem że o czymś zapomniałem
Jest w kodzie dużo literówek (np. highlited zamiast highlighted) i kilka metod napisanych po polskiemu. Ja bym to jak najszybciej wyeliminował, bo czym później tym większy będzie problem ze zmianą tego.
Taka moja mała uwaga - Localization to chyba nie jest najlepsza nazwa na coś, co można nazwać po prostu “City/Place”. Moje pierwsze i oczywiste skojarzenie dla terminu “Localization” dotyczyło tłumaczeń/wersji językowych.
A na poważniej o co chodzi z tymi metodami column w ContactHandler? 
