phd student now has email change button #1767
Conversation
|
Mało znane fakty: nie wszyscy użytkownicy systemu są studentami lub pracownikami – jest też kategoria Warto też będzie zostawić komentarz w kodzie, bo ktoś jeszcze zachce to w przyszłości "uprościć" w jedną lub w drugą stronę… |
| @@ -63,7 +63,7 @@ <h2>Moje dane</h2> | |||
| </table> | |||
|
|
|||
| <a class="btn btn-light" href="{% url 'password_change' %}" role="button">Zmień hasło</a> | |||
| {% if not user.student %} | |||
| {% if user.employee or is_external_contractor %} | |||
There was a problem hiding this comment.
Zostawiłbym też or not user.student żeby pozostawić dotychczasowe zachowanie dla hipotetycznych użytkowników nie będących w żadnej z tych trzech grup (bez tracenia czasu na dalsze zastanawianie się, czy oni w ogóle istnieją).
| @@ -159,9 +159,11 @@ def my_profile(request): | |||
| consultations). | |||
| """ | |||
| semester = Semester.get_upcoming_semester() | |||
| is_ext_contractor = is_external_contractor(request.user) | |||
There was a problem hiding this comment.
| is_ext_contractor = is_external_contractor(request.user) | |
| is_external_contractor = is_external_contractor(request.user) |
Jeszcze nie jest tak źle z długością naszych identyfikatorów / wierszy kodu.
closes #1754