ScriptyChris
commented
1 year ago Przeklikałem, działa. Fajnie, że dodaje lub usuwa punkty w zależności od tego, czy zamieni się komentarz na odpowiedź czy zrobi w drugą stronę.
Zauważyłem, że przy zamianie odpowiedzi na komentarz nadal pojawia się komunikat
Otrzeżenie: Ta zamiana nie może być cofnięta.
, który raczej jest już nieaktualny, skoro można zamienić komentarz na odpowiedź.
Przy okazji: dodałbym wysyłanie powiadomienia do autora przenoszonego posta - jeśli to jakiś głębszy problem lub brak w Q2A, to pewnie nadaje się na osobny PR.
Można też rozważyć (raczej w osobnym PR) opcję przeniesienia komentarza do dowolnego posta (pytania lub odpowiedzi), np.:
- z pytania do dowolnej odpowiedzi lub
- z dowolnej odpowiedzi do pytania lub
- z dowolnej odpowiedzi do innej odpowiedzi
awaluk
Od początku mamy dostępną opcję zamiany odpowiedzi na komentarz. Brakowało niestety opcji w drugą stronę, czyli aby można było komentarz zamienić na odpowiedź, np. gdy ktoś się pomyli lub w toku dyskusji okaże się, że jednak coś, co początkowo wydawało się tylko sugestią nagle powinno być odpowiedzią, bo rozwiązuje problem. Przygotowałem zmiany, które wprowadzają taką możliwość.
Całość jest analogiczna do opcji zamiany odpowiedzi na komentarz, przygotowany kod bazuje dokładnie na nim, więc miejscami może nie być najładniejszy. Na formularzu edycji komentarza pod pytaniem lub odpowiedzią pojawia się checkbox, który umożliwia dokonanie przeniesienia. Przeniesienia może dokonać każda osoba, która może edytować komentarz. Zabezpieczone jest także, aby autor nie mógł dokonać zmiany, gdy pytanie jest zamknięte, administracja nadal może wtedy to zrobić. W tej wersji nie będą przenoszone komentarze pod przenoszonym komentarzem, bo nie ma sposobu, aby automatycznie wykryć czy dany komentarz to kontynuacja wątku z tego komentarza i powinien być przeniesiony czy nie. Zrobienie wyboru komentarzy do przenoszenia byłoby jeszcze bardziej skomplikowane, więc póki co odpuściłem. Prościej można byłoby ewentualnie dać dodatkowy checkbox żeby przenieść wszystkie dalsze komentarze lub żadne. Jeśli zostałoby jak teraz to można jednak nadal przenieść komentarze pod odpowiedź - wystarczy zamienić je wszystkie na odpowiedź przy użyciu nowej funkcji, a później przy użyciu dotychczasowej zamiany na komentarz zmienić je ponownie w komentarze tyle, że tym razem do tej przeniesionej odpowiedzi. Trochę dookoła, ale do zrobienia, mniej wygodne gdy komentarzy będzie dużo.
Niestety nie udało mi się zrobić tego w postaci plugina, miałem problem już na samym początku, aby choćby dołożyć checkbox i potem go obsłużyć, stąd moja propozycja to po prostu modyfikacja kodu samego silnika. Fixes #263