lms icon indicating copy to clipboard operation
lms copied to clipboard

improvement: allow to choose unassigned and some user in user eventli…

Open interduo opened this issue 4 years ago • 8 comments

…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...

interduo avatar Mar 01 '21 07:03 interduo

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.

rafalpietraszewicz avatar Mar 12 '21 17:03 rafalpietraszewicz

@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.

interduo avatar Mar 12 '21 18:03 interduo

No dobrze regres, nie regres. PR wejdzie czy nie?

interduo avatar Mar 16 '21 14:03 interduo

puk-puk, backend był przygotowany na taki wybór już dawno @chilek

interduo avatar Mar 30 '21 13:03 interduo

???

interduo avatar Apr 01 '21 10:04 interduo

@chilek to może jeszcze to puścisz? Nie jest to zupełnie problematyczne a działa...

interduo avatar Apr 21 '21 15:04 interduo

puk-puk :)

interduo avatar Apr 29 '21 12:04 interduo

Czemu tego nie wrzucicie? Przecież to działa i wystarczy ta drobna zmiana na frontendzie?

interduo avatar Feb 05 '22 09:02 interduo