Plugin 2.0: testy

[JhnW]

No to tak...

@Gladin napisał w Plugin 2.0: testy:

Jeżeli to nie problem, proszę o zablokowanie opcji usunięcia wszystkich mistrzów gry z sesji.

Na pewno @dekline zrobi z tego notkę, ale kiedy to na tapet weźmiemy to nikt nie wie, nie jest to krytyczne. I tak, każdy moderator może wam sesje edytować więc problemu z tym nie ma. Również moderacja może całkowicie usuwać ludzi z sesji w przypadku pomyłek, MG ma tylko narzędzie aktywny/nieaktywny.

@Gladin napisał w Plugin 2.0: testy:

Może detal, ale czy forum testowe to na pewno ta sama wersja, którą mamy tutaj? Bo u mnie lewy panel nawigacyjny różni się jedną ikoną

To jak @santorine napisał, wersja silnika forum jest dokładnie taka sama. Różnią się czasami konfiguracją (w teorii staging powinno robić się jako mirror rel z nieprodukcyjną baza danych ale to za dużo roboty dla nas). W konfiguracji to forum różni się tym co jest klikalne plus ma doinstalowany nasz plugin. No i deploymenty na staging lecą automagicznie (bez backupa), a na produkcję robię je ręcznie.

[Gladin]

Testowaliście naprawianie takich sesji jak ta bez MG przed moderatorów?

[JhnW]

@Gladin napisał w Plugin 2.0: testy:

Testowaliście naprawianie takich sesji jak ta bez MG przed moderatorów?

Tak https://forum.staging.rolltelling.pl/rpg/session/11 To jest akurat dość trywialny scenariusz

[Gladin]

@jhnw Twoją idę było, aby w tytułach nie pojawiały się zbędne elementy, jak nazwa systemu. Czy jednak domyślne dodanie do nazwa wątków słów Komentarze, materiały nie byłoby dobrym pomysłem?

Zostawiłem domyślne, zarchiwizowałem sesję i ciężko odróżnić w archiwum co jest sesją, co komentarzami, a co materiałami. Pewnie po ilości wątków można się domyślić, ale... Można też zmieniać ręcznie, ale...

Rozwiązaniem alternatywnym (do dodawania info w nazwie) jest wstawienie nazw pól (rozgrywka, komentarze, materiały) w sekcji archiwum tak samo, jak są w sekcji głównej.

[JhnW]

@Gladin z poziomu widoku dowolnego tematu możesz wejść w widok całej sesji gdzie będziesz wiedzieć wszystkie jej tematy, wyświetlany tytuł sesji jest jednocześnie odnośnikiem. Imho to jest rozwiązanie, bez manipulacji nazwami tematów. Bardzo chciałem odejść od tych paskudnych tagów i dopisków do nazw tematów. Plus zawsze możesz nazwę tematu zmienić.

[Gladin]

To się sprawdza do momentu zarchiwizowania (inna rzecz co komu bardziej pasuje). W widoku archiwum, tego nie widać.
A przy okazji, moja sesja jest widoczna z głównej kategorii, tak ma być? Nie ma się pojawić nowa podkategoria archiwum i tam ma być widoczna? Bo dział future ma przygotowane archiwum (choć puste), a dział fantasy go nie ma.

[JhnW]

@Gladin napisał w Plugin 2.0: testy:

To się sprawdza do momentu zarchiwizowania (inna rzecz co komu bardziej pasuje). W widoku archiwum, tego nie widać.

W adalszym ciągu jak wejdziesz na sesję https://forum.staging.rolltelling.pl/topic/38/test-rozgrywki to po tytule możesz przejść w widok: https://forum.staging.rolltelling.pl/rpg/session/2

Co do tego, tak. Po prostu rozbiłem kategorie aby różne mechanizmy testować

[Gladin]

W takim razie to, że fantasy nie ma widocznych z głównego poziomu ani materiałów ani komentarzy też jest celowe? Że tylko rozgrywkę widać?

Moim zdaniem rozwiązanie z osobnym archiwum niż tak, jak to jest w fantasy jest lepsze.
Obecnie, gdy wchodzę w „Sesje fantasy” wyświetla mi też sesje zarchiwizowane i nie widać, które są aktywne, a które nie.

[Gladin]

Przy edycji gracza w sesji, następujące kroki:

1. wchodzę w edycję,
2. Edytuję jako kto gra (zmieniam nazwę, nie zatwierdzam przyciskiem)
3. Wybieram awatara z dysku, zatwierdzam.

Wgrywany jest awatar, nie jest zmieniana nazwa gracza. Drobnostka, ale upierdliwe, że trzeba pamiętać, aby osobno zmieniać nazwę, a osobno wgrywać obraz. Nie ma żadnej opcji, aby za jednym razem zmienić jedno i drugie - przynajmniej ja nie znalazłem.

[JhnW]

@Gladin napisał w Plugin 2.0: testy:

W takim razie to, że fantasy nie ma widocznych z głównego poziomu ani materiałów ani komentarzy też jest celowe? Że tylko rozgrywkę widać?

To jest po prostu skrócona forma widoku działu. Gdybyś poklikał trochę to byś zauważył, kliknięcie do środka działu ma guzik 3 więcej kategorii, nawet na Twoim screenie. Po prostu w tych sesjach testowałem bardziej kompaktowy widok, imho wygodniejszy. Wciąż realnie można wejść do każdego działu bez użycia widoku sesji.

[JhnW]

@Gladin napisał w Plugin 2.0: testy:

Wgrywany jest awatar, nie jest zmieniana nazwa gracza. Drobnostka, ale upierdliwe, że trzeba pamiętać, aby osobno zmieniać nazwę, a osobno wgrywać obraz. Nie ma żadnej opcji, aby za jednym razem zmienić jedno i drugie - przynajmniej ja nie znalazłem.

Nie ma ma i raczej nie będzie, w łatwy sposób. Jest to niestety limitacja techniczna gdy wchodzimy bezpośrednio we flow NodeBB, jak to kiedyś ogarniałem to okna się psuły. Może jest już lepiej. Można na kiedyś zrobić notkę @dekline

[Gladin]

@JhnW napisał w Plugin 2.0: testy:

Gdybyś poklikał trochę to byś zauważył

Widziałem, używałem (zanim napisałem poprzedni wpis) i uważam póki co to za mniej wygodne. Jednak po co w takim razie pozostaje widoczna „rozgrywka”, skoro resztę usuwamy? Ja to testuję z poziomu komputera i dla mnie nie ma różnicy czy widzę jeden poddział czy cztery. Z komórki nie planuję testować, zostawiam to innym.

@JhnW napisał w Plugin 2.0: testy:

Nie ma ma i raczej nie będzie, w łatwy sposób.

Dorobienie okna ostrzegającego o porzucaniu niezapisanych zmian będzie problemem?

Ew. opcji „zapisz zmiany i nie zamykaj”? Wtedy można zmienić nazwę i ją zapisać, a następnie przejść do awatara.

[JhnW]

Okno można, ale też nie jest to wysoki priorytet obecnie. Problem jest modyfikowanie flow samego edytora avatarów. My tak naprawdę używamy dokładnie tego samego edytora którego używa bazowy silnik (nawet kiedyś im nań buga zakładałem bo wbrew obietnicom, nie dało się tego używać customowo).

[Gladin]

Cóż... nie mogę oprzeć się wrażeniu, że ten silnik to jest jednak słaby...

[Zell]

Ja nie mogę się oprzeć wrażeniu, że część problemów to małe bzdurki. Co za różnica, że od razu się nie da wgrać ava i zmienić nazwy? Życia nie stracimy musząc dwie rzeczy zrobić. Co do silnika - to MY wymyśliliśmy sobie panel i granie w rpg. To silnik na fora dyskusyjne, nie mylcie potrzeb w jakim był robiony.

[JhnW]

@Gladin napisał w Plugin 2.0: testy:

Cóż... nie mogę oprzeć się wrażeniu, że ten silnik to jest jednak słaby...

W projektach programostycznych bardzo rzadko jest tak, że czegoś się nie da. Jest tylko pytanie za ile i na kiedy. Tak to też wygląda z pluginem, robiliśmy go z Santo po godzinach. Ale to nie jest jakiś nasz projekt życia czy drugi tetat. Stąd musimy balansować co robimy oraz jak - to drugie to kwestia utrzymania forum. Robienie pewnych rzeczy inaczej dokłada pracy przy każdym update silnika. Niech przykładem będzie umiejscowienie spisu sesji w profilu, obecnie dodanie dużej zakładki na to kosztowałoby nas sprawdzanie kompatybilności co update. Alternatywą jest użycie widgetu w głównym widoku, co zrobiliśmy i jest tańsze.

Stąd proszę nie bądź rozgoryczony, że dla pierdół które ani często używane nie będą, ani nie psują nic mamy priorytet "jak się komuś będzie nudzić to doda". Tym bardziej, że nie są to rzeczy które powinny opóźniać wydanie.

[Gladin]

Zwróćcie proszę uwagę, że mój zarzut był wobec słabości silnika, któremu z punktu widzenia mnie jako użytkownika brakuje wielu bardzo prostych funkcji, które po prostu powinny być. W końcu od dawna jesteśmy w XXI wieku. Właśnie te małe bzdurki świadczą o jakości. To nie jest zarzut wobec panelu sesji. I opinia nie opiera się tylko i wyłącznie na problemie z awatarem / nazwą postaci.

To nie był zarzut wobec waszej pracy czy pluginu.

[JhnW]

Nie znam twoich zarzutów w materii silnika, 8 nie wiem czemu jest o tym mowa w temacie plugin.

[Dekline]

Zrobiłem zestawienie tematów

Nawet tych wyjaśnionych - z zastrzeżeniem że dopiero od czasu gdy przejąłem obowiązki. Nie wszystko przeniosłem do Githuba, ponieważ:

Proszę o odpowiedź w wątkach:

• Widoczność podkategorii [Zell]
• Znaczenie statusu aktywny/nieaktywny [Santo]

Proszę o opinie innych osób w wątkach

• Powiadomienie o dodaniu użytkownika do sesji [Santo]
• Zbiorcze dodawanie i usuwanie użyszkodników[Santo]

Przypominam że Disco nie jest dobrym miejscem na zgłaszanie czegokolwiek. Ja tam zaglądam, ale w ciągu dnia, kiedy wchodzić nie mogę, potraficie zalać ściana tekstu z którego nic nie wynika, po 90% to dziamdziolenie z którego post factum ciężko się cokolwiek wyciąga.

No i macie jeszcze to, nie będę miał wymówiki że nie zauważyłem:
https://docs.google.com/forms/d/e/1FAIpQLSc6A4OLRFuuMrY69rczHbzxEpL2NAmruyb4lhACyTJsKDIb7g/viewform?usp=header

Zestawienie - nieprzesunięte na gita (stan na 29.03)

Widoczność podkategorii [Zell]

• Jak dla mnie podkategorie nie muszą być widoczne, jeżeli już są te tematy w odpowiednich sesjach
• Nie da się ich całkiem ukryć, produkcyjnie będą pewnie jak w dziale https://forum.staging.rolltelling.pl/category/8/sesje-fantasy czyli tylko 1 podkategoria widoczna i reszta rozwijana To jest rzecz łatwa do ogarnięcia. [Arbuz]
• Ok, ale jak wejdziesz już w jakąś kategorię to nie możesz wrócić, bo lista zwijania znika. CZzy ja czegoś nie rozumiem?[Dek]

Powiadomienie o dodaniu użytkownika do sesji [Santo]

• Czy dodanie użytkownika do sesji nie powinno stworzyć powiadomienia, że został tam dodany?
• Nie wiem czy nie większy do systemu powiadomień [Arbuz]
• Podbijam Arbuza [Dek]

Zbiorcze dodawanie i usuwanie użyszkodników[Santo]

• Dodawać użytkowników do sesji zawsze można pojedynczo. Czy nie byłoby lepiej, gdyby można było w pojedynczym dialogu dodawać wielu uczestników od razu, czyli zaznaczać ich, a potem odhaczać, że są w sesji? Coś w stylu https://github.com/codeshackio/multi-select-dropdown-js . Podobnie usuwanie.
• @dekline może dać notkę, ale według mnie jest to P5, to nie jest operacja tak częsta aby ją trzeba było kosztem innych rzeczy ulepszać. [Arbuz]
• To mam 5 kategori? Nie widziałem. Ale podbijam [Dek]

Usuwania Avka przez inne osoby[Santo]

• Usuwanie przesłanego zdjęcia w sesji (avatara) dla innych użytkowników nie działa. Tutaj dwa tematy: skoro jestem MG, to czy nie powinienem mieć absolutnej kontroli nad graczami? I drugi aspekt tego: założywszy, że tylko gracze ogarniają swoje avatary, to po co pokazywać kontrolkę, z której nie mogę skorzystać?
• Wydaje się mi, że to działa. Zell mi jako MG ustawiła np. w sesji avatar. Przed chwilą sprawdziłem i usuwanie jako drugi MG, nie-admin też śmiga.
• Zostawię to na później, chyba nie jest krytyczne, a mi tez sie zdaje ze działa [Dek]

Znaczenie statusu aktywny/nieaktywny [Santo]

• Od strony UX: dodać krótkie wyjaśnienie, co oznacza oznaczenie "Aktywny" i w jaki sposób wpływa na grę.
• W zasadzie nie wpływa Na przyszłość do statystyk [Arbuz]
• Santo, myślałeś o pływającym linku co wyskakuje notatka co to oznacza? [Dek]

Blokada usunięcia wszystkich MG [Gladin]

• zablokowanie opcji usunięcia wszystkich mistrzów gry z sesji.
• Na pewno @dekline zrobi z tego notkę, ale kiedy to na tapet weźmiemy to nikt nie wie, nie jest to krytyczne. I tak, każdy moderator może wam sesje edytować więc problemu z tym nie ma. Również moderacja może całkowicie usuwać ludzi z sesji w przypadku pomyłek, MG ma tylko narzędzie aktywny/nieaktywny. [Arbuz]
• TLDR; Było testowane, obsługa może sesje naprawić, ale feature dodamy na kiedyś tam [Dek]

Wgrywanie Avka i zmiana nazwy za jednym zamachem [Gladin]

• Przy edycji gracza w sesji, następujące kroki:

  1. wchodzę w edycję,
  2. Edytuję jako kto gra (zmieniam nazwę, nie zatwierdzam przyciskiem)
  3. Wybieram awatara z dysku, zatwierdzam.

  Wgrywany jest awatar, nie jest zmieniana nazwa gracza. Drobnostka, ale upierdliwe, że trzeba pamiętać, aby osobno zmieniać nazwę, a osobno wgrywać obraz. Nie ma żadnej opcji, aby za jednym razem zmienić jedno i drugie - przynajmniej ja nie znalazłem.

• Nie ma ma i raczej nie będzie, w łatwy sposób. Jest to niestety limitacja techniczna gdy wchodzimy bezpośrednio we flow NodeBB, jak to kiedyś ogarniałem to okna się psuły. Może jest już lepiej. Można na kiedyś zrobić notkę @dekline [Arbuz]

• TLDR Blokada lub monit o braku zapisania zmian przy braku takiej operacji [Gladin]

Odrzucone / załatwione ad-hoc / Zapisane na tablicy github

Więcej niż jeden wątek każdego typu w jednej sesji [Gladin]

• Użytkownik może już dodać tylko jeden wątek sesji w sesji. Chcemy mieć możliwość dodawania kolejnych.

Ustaw tagi dla każdego wątku w sesji jednym kliknięciem [Gladin]

• Użytkownik musi już dodawać tagi sesji osobno do każdego wątku w sesji. Chcemy dodawać je zbiorczo.

MG nie może usuwać użyszkodnika z sesji [Santo]

• Kiedy usuwam użytkownika z sesji (w przypadku testowego forum, jest to sesja "Cienie Srebrnego Miasta"), dostaję komunikat, że nie mam uprawnień. Bez sensu - jako MG powinienem mieć uprawnienia.
• MG powinien używać narzędzi aktywny i nieaktywny, nie chcemy bez śladu usuwać graczy którzy odeszli, po to mamy daty początku i końca aktywności na przyszłość. Usuwanie z sesji jest dla moderacji gdyby komuś się dodało przez przypadek. Ale to, że widzisz ten guzik to jest bug. Poprawione. [Arbuz]

Śmieci po usunięciu użyszkodnika [Santo]

• Usunięcie oznaczenia użytkownika jako "aktywny" zostawia tekst "rpg-invactive-label".
• Poprawione [Arbuz]

Skalowanie panelu sesji [Santo]

• Mam także pewne wątpliwości co do skalowania panelu sesji. Wydaje się, że po przekroczeniu siedmiu użytkowników, kolejna belka awatarów pokazuje się krzywo, pewnie mniej przy mniejszych rozdzielczościach.
• Poprawiłem, ogólnie nie chodziło chyba nawet o 7 tylko o nieparzystą liczbę graczy, na mobilkach brzydko wyglądało. Ogarnięte. [Arbuz]

Automatyczna zmiana nazwy tematu [Zell]

• Jeżeli chce się zmienić tytuł automatycznie stworzonego tematu to nie aktualizuje się nazwa póki nie stworzy się samemu nowego tematu danego typu. Wtedy pokażą się dobre nazwy.
• Bo on nie zmienia nazw [Arbuz]

Widoczność belek z tytułami [Zell]

• "Belki" z tytułami kategorii pojawiają się tylko, gdy danych kategorii jest więcej niż jedna. Lepiej by wyglądało jakby były one zawsze
• Tematy domyślnie mają nazwę sesji. Po prostu jeśli w danym typie jest tylko 1 temat (np. jeden temat rozgrywki) to dostaje w widoku sesji nazwę Rozgrywka. Nie zmienia to jego właściwej nazwy. Dopiero przy mnogości tematów dodajemy etykiety i wyświetlamy prawdziwe nazwy. [Arbuz]

Czy forum testowe to na pewno ta sama wersja? [Gladin]

• Lewy panel nawigacyjny różni się jedną ikoną (pierwsza o góry)
• Z tego, co się orientuję, jest to kwestia konfiguracji danej instancji forum, a nie jego wersji. To znaczy, ściśle w technicznym rozumieniu, to nie jest ta sama wersja, bo to wersja z nową wersją pluginu, ale ikona nie jest wiążąca, ponieważ funkcjonalność nie jest uzależniona od wersji forum tylko tego, w jaki sposób dane forum zostało skonfigurowane. [Santo]
• to ikonka dodana przez @jhnw dla mnie, bardzo przydatna raz jeszcze dzięki za dodanie! [Wired]
• To jak @santorine napisał, wersja silnika forum jest dokładnie taka sama. Różnią się czasami konfiguracją (w teorii staging powinno robić się jako mirror rel z nieprodukcyjną baza danych ale to za dużo roboty dla nas). W konfiguracji to forum różni się tym co jest klikalne plus ma doinstalowany nasz plugin. No i deploymenty na staging lecą automagicznie (bez backupa), a na produkcję robię je ręcznie. [Arbuz]
• TLDR: Ogólnie tak [Dek]

Brak tagów - rozróżnianie tematów aktywnych od archiwum [Gladin]

• TLDR; Można wejść we wszystkie tematy związane z sesja poprzez klikniecie nazwy sesji. Niektore działy mają aktualnie inne ustawienia w celach testów tychże

Limit graczy w sesji [Santo]

• Wydaje się, że nie ma limitu na to, ile osób może uczestniczyć w sesji. Pytanie, czy to zamierzone, ponieważ w teorii całe forum może grać w jedną sesję.
• Myślę że nie powinno być limitu, nikt raczej nie będzie grał w 50ciu. A jesli juz to dla beki albo rozwalenia serwera, to i tak to zdejmiemy [Dek]

[Santorine]

@Dekline napisał w Plugin 2.0: testy:

Powiadomienie o dodaniu użytkownika do sesji [Santo]
Zbiorcze dodawanie i usuwanie użyszkodników[Santo]

Tematy: powiadomienie, zbiorcze dodawanie i usuwanie mają mały priorytet. W zasadzie nie były tyle problemami dla mnie, co po prostu głośno zastanowiłem się nad ulepszeniem UX.

@Dekline napisał w Plugin 2.0: testy:

Usuwania Avka przez inne osoby[Santo]

Usuwanie avatara przez inne osoby prawie na pewno działa. Nie potrafiłem zreprodukować tego błędu potem, więc może to był jakiś lag lub cokolwiek. Sądzę, że temat jest prawie na pewno zamknięty.

@Dekline napisał w Plugin 2.0: testy:

Santo, myślałeś o pływającym linku co wyskakuje notatka co to oznacza? [Dek]

Zdecydowanie najlepsza opcja ogarnięcia tego.