lms
lms copied to clipboard
improvement: allow to choose unassigned and some user in user eventli…
…stfilters
Likwidacja drobnego regresu związanego z przeróbką filtru Użytkownik w Terminarzu i widoku Plan. PR daje możliwość wyboru "nieprzypisany" wraz z użytkownik1,użytkownik2...
Likwidacja drobnego regresu związanego z przeróbką filtru Użytkownik w Terminarzu i widoku Plan. PR daje możliwość wyboru "nieprzypisany" wraz z użytkownik1,użytkownik2...
@interduo nie określiłbym tego mianem 'regresu', bo od początku do wyboru było 'nieprzypisany' LUB 'użytkownicy'. Użycie słowa 'regres' wskazuje, że w międzyczasie coś nie działało jak trzeba, a w tym przypadku wcale tak nie jest. Niepotrzebnie nadużywasz tego słowa, bo na hasło 'regres' trzeba sprawdzać czy przypadkiem nie było i nie ma problemów na wszystkich stabilnych wersjach. Zatem skoro użyłeś słowa 'regres' to daj też odpowiednie etykiety do zgłoszenia. Czy to ma być 'master-only' czy też do gałęzi stabilnych.
@interduo nie określiłbym tego mianem 'regresu', bo od początku do wyboru było 'nieprzypisany' LUB 'użytkownicy'.
Przepisałem kiedyś część metody właśnie po to by móc taki wybór dokonywać .... ale fakt pamięć mnie trochę zawiodła bo było to nie w tym miejscu. Także określenie regres w tym wypadku jest nietrafne.
Zatagowane.
No dobrze regres, nie regres. PR wejdzie czy nie?
puk-puk, backend był przygotowany na taki wybór już dawno @chilek
???
@chilek to może jeszcze to puścisz? Nie jest to zupełnie problematyczne a działa...
puk-puk :)
Czemu tego nie wrzucicie? Przecież to działa i wystarczy ta drobna zmiana na frontendzie?