Rapotek : wielkie WOW!

Tu piszemy na tematy związane z serwisem Opencaching.pl

Moderator: Moderatorzy

Awatar użytkownika
deg
Rada Techniczna
Posty: 1093
Rejestracja: sobota 07 grudnia 2013, 04:31
Podziękował;: 217 razy
Otrzymał podziękowań: 492 razy

Rapotek : wielkie WOW!

Post autor: deg »

Dostałem właśnie mailem powiadomienie o zmianie kodu OC PL i... spadłem z krzesła niemal!

Okazuje się, że Rapotek - sam od siebie - przygotował poprawkę kodu łatającą pewien problem. Przedstawił ją następnie do zatwierdzenia (fachowo nazywa się to Pull Request), RT w osobie kojotów zatwierdziło i poprawka już działa na wszystkich serwerach opartych o kod OC PL! Szok!

Brawo Rapotek!!!

https://github.com/opencaching/opencaching-pl/pull/1300
Za ten post autor deg otrzymał podziękowania (total 2):
rapotekkrcr
"Kto ma rację dzień wcześniej od innych, ten przez dobę uchodzi za idiotę" Antoni de Rivarol
Obrazek

rapotek
Wyjadacz
Posty: 212
Rejestracja: piątek 26 lutego 2016, 11:54
Podziękował;: 113 razy
Otrzymał podziękowań: 90 razy

Re: Rapotek : wielkie WOW!

Post autor: rapotek »

Ale przecież to był drobiazg, więcej zachodu było ze zrobieniem Pull Request (musiałem się tego nauczyć) niż poprawienie dosłownie jednej instrukcji w kodzie. Bez przesady, przecież wszyscy tak robicie i to nie raz i nie od dzisiaj.

W dodatku nie wiem jeszcze, czy działa, bo generowanie dynamicznych danych chyba z crona idzie.
Ostatnio zmieniony czwartek 09 listopada 2017, 15:14 przez rapotek, łącznie zmieniany 1 raz.

Awatar użytkownika
deg
Rada Techniczna
Posty: 1093
Rejestracja: sobota 07 grudnia 2013, 04:31
Podziękował;: 217 razy
Otrzymał podziękowań: 492 razy

Re: Rapotek : wielkie WOW!

Post autor: deg »

Rapotek - dopiero po napisaniu mojego posta na forum zauważyłem, że przeprowadziłeś małe śledztwo, znalazłeś błąd (wcale nie taki oczywisty), poprawnie wrzuciłeś komentarz do issue, a nawet w PR tytułu użyłeś tak, by issue samo się zamknęło po merge! Pełen profesjonalizm!
"Kto ma rację dzień wcześniej od innych, ten przez dobę uchodzi za idiotę" Antoni de Rivarol
Obrazek

kojoty
Rada Techniczna
Posty: 334
Rejestracja: środa 26 listopada 2014, 14:14
Podziękował;: 14 razy
Otrzymał podziękowań: 202 razy

Re: Rapotek : wielkie WOW!

Post autor: kojoty »

i takich ludzi trzeba nam więcej!

(@rapotek - podeślij nam próbkę swojego dna - będziemy Cię klonować)

rapotek
Wyjadacz
Posty: 212
Rejestracja: piątek 26 lutego 2016, 11:54
Podziękował;: 113 razy
Otrzymał podziękowań: 90 razy

Re: Rapotek : wielkie WOW!

Post autor: rapotek »

OK, już wiem że zadziałało :D

I napiszę tak, kończąc chyba temat: tyle razy było pisane, że przecież każdy może tworzyć kod i naprawiać błędy. To skąd nagle takie WOW, gdy ktoś zobaczył coś szybkiego do naprawy i postanowił się tym zająć? Powtórzę jeszcze raz: każdy kto orientuje się choć trochę w PHP, js, bazach danych, grafice itp. - a takich ludzi jest całkiem sporo - może spróbować sił.

I oczywiście dzięki @kojoty za instrukcje jak się w tym poruszać :)

kojoty
Rada Techniczna
Posty: 334
Rejestracja: środa 26 listopada 2014, 14:14
Podziękował;: 14 razy
Otrzymał podziękowań: 202 razy

Re: Rapotek : wielkie WOW!

Post autor: kojoty »

każdy może, ale nie każdy chce :) - często nam się to nie zdarza więc WOW :)

triPPer
Rada Techniczna
Posty: 223
Rejestracja: środa 23 października 2013, 22:03
Podziękował;: 51 razy
Otrzymał podziękowań: 129 razy

Re: Rapotek : wielkie WOW!

Post autor: triPPer »

Dzięki @raportek :)
Łap chwile ...

ODPOWIEDZ