From 3aa84cd04cafb1bcdfb3f0019f52dc7441c65f30 Mon Sep 17 00:00:00 2001 From: Timofey Mazurov Date: Tue, 30 Mar 2021 00:29:47 +0300 Subject: [PATCH 1/3] Fixed role display on profile and admin pages --- main/migrations/0015_auto_20210330_0007.py | 18 ++++++++++++++++++ main/templates/pages/adm_ruleset.html | 7 ++++++- main/templates/pages/profile.html | 8 +++++++- main/views.py | 4 +++- 4 files changed, 34 insertions(+), 3 deletions(-) create mode 100644 main/migrations/0015_auto_20210330_0007.py diff --git a/main/migrations/0015_auto_20210330_0007.py b/main/migrations/0015_auto_20210330_0007.py new file mode 100644 index 0000000..91398ba --- /dev/null +++ b/main/migrations/0015_auto_20210330_0007.py @@ -0,0 +1,18 @@ +# Generated by Django 3.1.6 on 2021-03-29 21:07 + +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ('main', '0014_auto_20210314_1455'), + ] + + operations = [ + migrations.AlterField( + model_name='unassignedticket', + name='status', + field=models.IntegerField(choices=[(0, 'Снят с пользователя, перенесён в буферную группу'), (1, 'Авторство восстановлено'), (2, 'Пока нас не было, тикет испарился из буферной группы. Дополнительные действия не требуются'), (3, 'Тикет уже был закрыт. Дополнительные действия не требуются'), (4, 'Тикет решён. Записан на пользователя с почтой SOLVED_TICKETS_EMAIL')], default=0), + ), + ] diff --git a/main/templates/pages/adm_ruleset.html b/main/templates/pages/adm_ruleset.html index 92686f1..98700e7 100644 --- a/main/templates/pages/adm_ruleset.html +++ b/main/templates/pages/adm_ruleset.html @@ -56,7 +56,12 @@ {{ user.name }} {{ user.user.email }} - {{ user.role }} + {% if user.custom_role_id == ZENDESK_ROLES.engineer %} + engineer + {% elif user.custom_role_id == ZENDESK_ROLES.light_agent %} + light_agent + {% endif %} + {% endfor %} diff --git a/main/templates/pages/profile.html b/main/templates/pages/profile.html index a0f21f9..1dd6005 100644 --- a/main/templates/pages/profile.html +++ b/main/templates/pages/profile.html @@ -37,7 +37,13 @@
Электронная почта {{ profile.user.email }}

-
Текущая роль {{ profile.role }}
+
Текущая роль + {% if profile.custom_role_id == ZENDESK_ROLES.engineer %} + engineer + {% elif profile.custom_role_id == ZENDESK_ROLES.light_agent %} + light_agent + {% endif %} +
diff --git a/main/views.py b/main/views.py index a8b3af5..1a2ad73 100644 --- a/main/views.py +++ b/main/views.py @@ -104,7 +104,8 @@ def profile_page(request: WSGIRequest) -> HttpResponse: update_profile(user_profile) context = { 'profile': user_profile, - 'pagename': 'Страница профиля' + 'pagename': 'Страница профиля', + 'ZENDESK_ROLES': ZENDESK_ROLES, } return render(request, 'pages/profile.html', context) @@ -203,6 +204,7 @@ class AdminPageView(LoginRequiredMixin, PermissionRequiredMixin, FormView): users = get_list_or_404( UserProfile, role='agent') context['users'] = users + context['ZENDESK_ROLES'] = ZENDESK_ROLES context['engineers'], context['light_agents'] = count_users(get_users_list()) context['licences_remaining'] = max(0, ZENDESK_MAX_AGENTS - context['engineers']) return context # TODO: need to get profile page url From 99979858f07877db6ef9aede17c2744b30e4a75e Mon Sep 17 00:00:00 2001 From: Timofey Mazurov Date: Tue, 30 Mar 2021 00:30:38 +0300 Subject: [PATCH 2/3] Deleted main page logging example --- main/views.py | 5 ----- 1 file changed, 5 deletions(-) diff --git a/main/views.py b/main/views.py index 1a2ad73..1bbe593 100644 --- a/main/views.py +++ b/main/views.py @@ -163,11 +163,6 @@ def work_become_engineer(request): def main_page(request): - """ - Отображение логгирования на главной странице - """ - logger = logging.getLogger('main.index') - logger.info('Index page opened') return render(request, 'pages/index.html') From 17328650c8aeb68ce8268208a88a33055fceb255 Mon Sep 17 00:00:00 2001 From: Timofey Mazurov Date: Tue, 30 Mar 2021 00:45:32 +0300 Subject: [PATCH 3/3] Fixed control page role changing --- main/extra_func.py | 2 ++ main/migrations/0016_merge_20210330_0043.py | 14 ++++++++++++++ 2 files changed, 16 insertions(+) create mode 100644 main/migrations/0016_merge_20210330_0043.py diff --git a/main/extra_func.py b/main/extra_func.py index b87ac27..077615f 100644 --- a/main/extra_func.py +++ b/main/extra_func.py @@ -150,6 +150,8 @@ def update_role(user_profile: UserProfile, role: str) -> UserProfile: zendesk = ZendeskAdmin() user = zendesk.get_user(user_profile.user.email) user.custom_role_id = role + user_profile.custom_role_id = role + user_profile.save() zendesk.admin.users.update(user) diff --git a/main/migrations/0016_merge_20210330_0043.py b/main/migrations/0016_merge_20210330_0043.py new file mode 100644 index 0000000..efb1d45 --- /dev/null +++ b/main/migrations/0016_merge_20210330_0043.py @@ -0,0 +1,14 @@ +# Generated by Django 3.1.6 on 2021-03-29 21:43 + +from django.db import migrations + + +class Migration(migrations.Migration): + + dependencies = [ + ('main', '0015_auto_20210330_0007'), + ('main', '0015_auto_20210321_1600'), + ] + + operations = [ + ]