Rapotek : wielkie WOW!

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

Moderator: Moderatorzy

Awatar użytkownika
deg
Forumator
Posty: 1054
Rejestracja: sobota 07 grudnia 2013, 04:31
Podziękował: 196 razy
Otrzymane podziękowania: 469 razy

Rapotek : wielkie WOW!

Post autor: deg » czwartek 09 listopada 2017, 15:03

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
Użytkownicy, którzy podziękowali autorowi deg za post (razem 2):
rapotekkrcr
"Kto ma rację dzień wcześniej od innych, ten przez dobę uchodzi za idiotę" Antoni de Rivarol
Obrazek

rapotek
Stały bywalec
Posty: 183
Rejestracja: piątek 26 lutego 2016, 11:54
Lokalizacja: Pomorze

Re: Rapotek : wielkie WOW!

Post autor: rapotek » czwartek 09 listopada 2017, 15:06

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
Forumator
Posty: 1054
Rejestracja: sobota 07 grudnia 2013, 04:31

Re: Rapotek : wielkie WOW!

Post autor: deg » czwartek 09 listopada 2017, 15:13

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: 317
Rejestracja: środa 26 listopada 2014, 14:14

Re: Rapotek : wielkie WOW!

Post autor: kojoty » czwartek 09 listopada 2017, 15:21

i takich ludzi trzeba nam więcej!

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

rapotek
Stały bywalec
Posty: 183
Rejestracja: piątek 26 lutego 2016, 11:54
Lokalizacja: Pomorze

Re: Rapotek : wielkie WOW!

Post autor: rapotek » czwartek 09 listopada 2017, 17:28

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: 317
Rejestracja: środa 26 listopada 2014, 14:14

Re: Rapotek : wielkie WOW!

Post autor: kojoty » czwartek 09 listopada 2017, 17:36

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

Re: Rapotek : wielkie WOW!

Post autor: triPPer » czwartek 09 listopada 2017, 21:47

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

ODPOWIEDZ