Rapotek : wielkie WOW!

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

Moderator: Moderatorzy

Awatar użytkownika
deg
Forumator
Posty: 648
Rejestracja: sobota 07 grudnia 2013, 04:31
Podziękował: 126 razy
Otrzymane podziękowania: 222 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
Obrazek

rapotek
Bywalec
Posty: 74
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: 648
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!
Obrazek

kojoty
Rada Techniczna
Posty: 135
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
Bywalec
Posty: 74
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: 135
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: 205
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 ...

areckis
Stały bywalec
Posty: 193
Rejestracja: piątek 14 sierpnia 2015, 12:40

Re: Rapotek : wielkie WOW!

Post autor: areckis » czwartek 09 listopada 2017, 22:04

Szacun rapotek. :) A temat potrzebny, bo niby wiadomo, że każdy może być takim rapotkiem ;) ale nie każdy ma odwagę...
rapotek pisze:I oczywiście dzięki @kojoty za instrukcje jak się w tym poruszać :)
Więc może by się przydała taka instrukcja co i jak dla innych, może w wiki?
Sekeszuje
----
Geocaching jest to gra terenowa polegająca na szukaniu skrzynek ze "skarbami".
Trolling od odłowienia ryb - "zarzucanie haczyka”, poruszając kontrowersyjny temat, często niepotrzebnie, aby wywołać kłótnię.

ODPOWIEDZ