[Feature] [Plugin] Automatyczne wysyłanie wiadomości PW do nowo zarejestrowanego użytkownika #149
Conversation
|
Co do kodu chyba nie mam zastrzeżeń, wiele tu nie ma :) Nie bardzo widzę sens oznaczania wielkimi czerwonymi literami pola z id użytkownika, ale ok. Dodałbym przed wysłaniem sprawdzenie czy treść wiadomości nie jest pusta - jeśli nawet jest włączone wysyłanie a treść nie jest wpisana to raczej nie ma sensu. Wydaje się też, że należałoby sprawdzać czy user o podanym id istnieje, bo jeśli nie to co? Wiadomość przyjdzie jako "od nikogo" albo wcale się nie wyśle? Ponadto w ustawieniach pluginu nie da się już wyłączyć wysyłania - jest włączone, odznaczam checbkoxa, biorę zapisz i nadal jest włączone. Samo wysłanie wiadomości działa. Wysyłana jest jednak tylko wiadomość na forum, użytkownik nie dostaje jej emailem - czy to dobrze czy źle, to sam nie wiem. Może dobrze, bo od razu po rejestracji i tak dostaje maila z powitaniem i linkiem do aktywacji konta. Źle bo można się pogubić jak to PW w końcu działa - czy wysyła informacje o nich mailem czy nie (a domyślnie wszystkie wysyła). Tak więc w sumie sam nie wiem. No i powiadomienie jak napisałeś nie działa, a to jest raczej kluczowy element całości, bo tak to nikt się o tej wiadomości nie dowie. Przeanalizowałbym jak one są wyciągane do wyświetlenia, to chyba tutaj, może to coś pomoże. Jeśli miałoby to wejść to chętnie pozbieram też sugestie jak chcemy nazwać konto wysyłające (czy po prostu "Forum Pasja Informatyki" lub "Forum" czy może coś bardziej wymyślnego) oraz jaka ma być finalnie treść wiadomości - żeby nie było ani za długo ani za krótko, już dyskusja była w #122 ale w sumie nic z niej nie wyniknęło. |
Mój pierwszy PR bez zastrzeżeń do kodu 😉
Tak jak jest napisane -
W sumie faktycznie, nie pomyślałem, jutro wrzucę poprawkę.
Z tego co pamiętam w takiej sytuacji wiadomość zostanie wysłana od użytkownika
To dziwne, wcześniej u mnie działało. Przyjrzę się temu.
Wg mnie prosto -
Treść jest raczej do przedyskutowania - czy wiadomość bardziej krótka, żeby zademonstrować działanie PW, czy może trochę dłuższa, ale przekazująca ważne informacje? |
|
Ja bym nadawał krótką wiadomość powitalną, ponieważ na Forum są już podane informacje dotyczące zasad korzystania i inne metadane: regulamin, FAQ, manifest, porady nie od parady, ekipa ninja, zasady przyznawania punktów. Dlatego w propozycji podanej w #122 wspomniał bym jeszcze o charakterze udzielanej pomocy i tyle: "Witaj na Forum Pasja Informatyki! Podstawowe odpowiedzi na pytania jak korzystać z forum znajdziesz w naszym dziale FAQ. Pamiętaj, że na forum skupiamy się głównie na pomocy w rozwiązaniu problemów programistycznych, a nie udzielaniu gotowych odpowiedzi na pytania. Powodzenia! :)" Na nazwę konta wysyłającego wg mnie odpowiednie będzie "Forum Pasja Informatyki", gdyż nazwa ta wskazuje na... nazwę forum, po prostu. :) |
|
Udało mi się znaleźć sposób na wysyłanie powiadomienia dla użytkownika. Plugin działa :) |
To nadal zachowuje się tak samo, wysyła mi puste wiadomości.
To zaraz po rejestracji wyrzuca że wystąpił jakiś błąd Q2A, czyli sprawdzania nadal nie ma. Może weryfikacja poprawnego id usera i nie pustej treści wiadomości powinna być już w momencie włączania pluginu w panelu - tylko w momencie poprawnego wypełnienia konfiguracji plugin zostanie włączony?
Tu też bez zmian. Powiadomienie przychodzi, fajnie. Ale coś jest z nim nie tak, bo gdy próbuję otworzyć listę powiadomień (klikam na "1") to kończy się tak: |
fixed
fixed
fixed
fixed Wszystko naprawione, powinno już działać, jak są jakieś błędy to dajcie znać. |
|
W działaniu od strony użytkownika jest ok, ale w panelu administracyjnym nie dostaję żadnej informacji o błędzie - gdy np. tylko włączę plugin i nie wpiszę treści ani id usera, to strona ustawień się jakby odświeża i nic. Ani się to nie zapisuje, ani nie ma błędu z wyjaśnieniem co się stało. Nadal nie mogę żadnym sposobem wyłączyć pluginu, czy tylko odznaczę opcję włączenia, czy też usunę wartości wszystkich pól, po zapisaniu strona się przeładowuje i wartości nadal są, plugin jest włączony. Podobnie gdy wpiszę id nieistniejącego użytkownika nie ma żadnego błędu, a strona się po prostu przeładowuje. |
|
Solved. |
|
Może to już będzie że się czepiam, ale jak widzę zielony komunikat to jestem przekonany, że to jest sukces, a tam "invalid bot id". Mylące, dałbym standardowo czerwony, jak mamy wszędzie indziej. Poza tym gdy próbuję zapisać ustawienia, a nie zaznaczę "enable plugin" to nie zapisuje się nic. Niby nie jest to wielki problem, ale to znowu jest mylące. Załóżmy, że przygotuję sobie treść, którą chcę wysyłać, ale jeszcze nie chcę włączyć, więc wpisuje, klikam zapisz i nic, nie zapisuje się, nie dostaję żadnego błędu. Rozumiałbym walidację w stylu gdy zaznaczę checkboxa, a nie wypełniłem treści albo id, to miałoby sens, ale wtedy też wypadałoby dać jakiś komunikat. Jak już z kolei włączę, później wyłączę i chcę edytować treść na wyłączonym to znów się nie da - klikam zapisz i nic się nie zapisuje dopóki nie włączę. Ja wiem że to tylko panel dla administratora, zmienione to będzie pewnie sporadycznie i jak teraz to dodam to będzie ok, ale jak wrócę za rok to się będę zastanawiał o co tu chodzi, że to się nie zapisuje. Niech to będzie jakoś minimalnie intuicyjne. |
| "update_uri": "https://github.com/CodersCommunity/forum.pasja-informatyki.local", | ||
| "min_q2a": "1.7", | ||
| "min_php": "5.3" | ||
| } No newline at end of file |
| 'qa_pw_event', | ||
| 'PW Event'); | ||
|
|
||
| ?> No newline at end of file |
There was a problem hiding this comment.
nie powinno być ?> oraz brak nowej linii na końcu pliku
event15
left a comment
event15
left a comment
There was a problem hiding this comment.
Aczkolwiek nie sprawdzałem tego lokalnie u siebie. Nie widzę nic rażącego w poprawkach.
Bez zmian Gdy wpiszę id bota i treść, a nie włączę plugina, to zapisuje się treść, id bota nie zapisuje się wcale bez żadnego komunikatu. Mam włączony plugin, wpisane id bota i treść. Wchodzę, zmieniam id na błędne, zapisuje. Dostaje komunikat (na zielono czyli nadal myli), że id jest błędne, więc się nie zapisuje i zostaje stare id, ale jednocześnie wyłącza plugin. Podtrzymuję to co napisałem komentarz wcześniej. Postaw się w roli użytkownika, jakbyś to Ty dostał ten plugin i korzystał z niego pierwszy raz:
Samo wysyłanie PW jest w porządku. Ale przyszło mi do głowy sprawdzenie jeszcze jednej rzeczy. Włączyłem plugin, wpisałem w ustawieniach id usera, którego następnie usunąłem i spróbowałem się zarejestrować. Prawdopodobnie taki przypadek u nas nie wystąpi, jednak wydaje się, że powinno to być zabezpieczone. Efekt to niestety niedziałająca rejestracja:
|
| $messageContent = qa_opt('sendPwMessageOnRegister_messageContent'); | ||
|
|
||
| if (empty($botId) || empty($messageContent)) { | ||
| if (empty($botId) || empty($messageContent) || [] === qa_db_user_get_userid_handles(qa_post_text('botId'))) { |
Tak jak w tytule; plugin ma za zadanie umożliwić userowi zapoznać się z systemem wiadomości prywatnych. Sama wiadomość jest ustalana w panelu administratora a także ID bota (usera od którego będzie szła wiadomość)
Ps. Wie ktoś może jak wysłać powiadomienie do usera? Trzeba chyba dodać jakiś wpis w qa_eventlog ale dodałem ich ok. 70 a żaden nie działa. Jakieś pomysły? ;)