[M] Zablokować możliwość oceniania zmian w stanie MERGED i ABANDONED

mobilecodereviewer / MobileCodeReviewer

0 stars 0 forks source link

[M] Zablokować możliwość oceniania zmian w stanie MERGED i ABANDONED #28

Closed fracz closed 9 years ago

fracz commented 9 years ago

Obecnie przy próbie oceny leci wyjątek - wysłałem stacktrace.

BTW - docierają te maile z dumpami do was?

liscju commented 9 years ago

Zrobione od wersji 0.3.8, przy próbie dodania komentarza jako użytkownik anonymous ,lub gdy status zmiany to merged lub abondoned wyświetla komunikat na ekranie o braku możliwości dodania komentarza

d00d171 commented 9 years ago

Maile docierają

fracz commented 9 years ago

Przy próbie dodania komentarza do linii faktycznie wyświetla się toast. Natomiast gwiazdką nadal mogę dodać komentarz (może zostać) oraz ocenić zmianę - wtedy nic się nie dzieje. Można by ukryć tam flagi albo przynajmniej pokazać "Unauthorized" tak jak jest to wyświetlane gdy nie mamy uprawnień.

liscju commented 9 years ago

Poprawione od wersji 0.3.19