From 1b60d4344e03e6be6086b2ba34d64041db99e6e6 Mon Sep 17 00:00:00 2001 From: jain Date: Sun, 9 Jul 2023 19:13:05 +0530 Subject: [PATCH] jira-15 top leadership API --- guardian/serializers.py | 32 ++++---------------------- guardian/views.py | 18 ++++----------- junior/admin.py | 11 ++++++++- junior/migrations/0008_juniorpoints.py | 28 ++++++++++++++++++++++ junior/models.py | 18 +++++++++++++++ 5 files changed, 65 insertions(+), 42 deletions(-) create mode 100644 junior/migrations/0008_juniorpoints.py diff --git a/guardian/serializers.py b/guardian/serializers.py index 1b6fa7e..a0a85a6 100644 --- a/guardian/serializers.py +++ b/guardian/serializers.py @@ -13,7 +13,7 @@ from account.serializers import JuniorSerializer from junior.serializers import JuniorDetailSerializer from base.messages import ERROR_CODE, SUCCESS_CODE from .utils import upload_image_to_alibaba -from junior.models import Junior +from junior.models import Junior, JuniorPoints class UserSerializer(serializers.ModelSerializer): """User serializer""" auth_token = serializers.SerializerMethodField('get_auth_token') @@ -181,37 +181,13 @@ class TaskDetailsSerializer(serializers.ModelSerializer): 'junior', 'task_status', 'is_active', 'created_at','updated_at'] - - - class TopJuniorSerializer(serializers.ModelSerializer): - total_points = serializers.SerializerMethodField() - - email = serializers.SerializerMethodField('get_auth') - first_name = serializers.SerializerMethodField('get_first_name') - last_name = serializers.SerializerMethodField('get_last_name') - - def get_auth(self, obj): - return obj.auth.username - - def get_first_name(self, obj): - return obj.auth.first_name - - def get_last_name(self, obj): - return obj.auth.last_name + junior = JuniorDetailSerializer() class Meta: - model = Junior - fields = ['id', 'email', 'first_name', 'last_name', 'phone', 'country_code', 'country_name', 'gender', 'dob', 'image', 'junior_code', 'guardian_code', 'referral_code', 'referral_code_used', 'is_active', 'is_complete_profile', 'passcode', 'is_verified', 'created_at', 'updated_at', 'total_points'] + model = JuniorPoints + fields = ['id', 'junior', 'total_task_points', 'created_at', 'updated_at'] - def get_total_points(self, obj): - junior_ids_with_total_points = self.context.get('junior_ids_with_total_points') - if junior_ids_with_total_points: - junior_id = obj.id - for item in junior_ids_with_total_points: - if item['junior'] == junior_id: - return item['total_points'] - return 0 class GuardianProfileSerializer(serializers.ModelSerializer): """junior serializer""" diff --git a/guardian/views.py b/guardian/views.py index eba6248..e0f2bef 100644 --- a/guardian/views.py +++ b/guardian/views.py @@ -10,7 +10,7 @@ from datetime import datetime, timedelta from .serializers import (UserSerializer, CreateGuardianSerializer, TaskSerializer, TaskDetailsSerializer, TopJuniorSerializer) from .models import Guardian, JuniorTask -from junior.models import Junior +from junior.models import Junior, JuniorPoints from junior.serializers import JuniorDetailSerializer from account.models import UserEmailOtp from .tasks import generate_otp @@ -139,23 +139,15 @@ class SearchTaskListAPIView(viewsets.ModelViewSet): serializer = TaskDetailsSerializer(paginated_queryset, many=True) return custom_response(None, serializer.data, response_status=status.HTTP_200_OK) + class TopJuniorListAPIView(viewsets.ModelViewSet): """Top juniors list""" serializer_class = TopJuniorSerializer permission_classes = [IsAuthenticated] - queryset = JuniorTask.objects.all() + queryset = JuniorPoints.objects.all() def list(self, request, *args, **kwargs): """fetch junior list those complete their task""" - junior_ids_with_total_points = JuniorTask.objects.filter(task_status=1) \ - .values('junior') \ - .annotate(total_points=Sum('points')) \ - .order_by('-total_points') - - junior_ids = [item['junior'] for item in junior_ids_with_total_points] - - juniors = Junior.objects.filter(id__in=junior_ids) - - serializer = self.get_serializer(juniors, many=True, context={'junior_ids_with_total_points': - junior_ids_with_total_points}) + junior_total_points = self.get_queryset().order_by('-total_task_points') + serializer = self.get_serializer(junior_total_points, many=True) return custom_response(serializer.data, response_status=status.HTTP_200_OK) diff --git a/junior/admin.py b/junior/admin.py index 87cd7d8..bb9ea49 100644 --- a/junior/admin.py +++ b/junior/admin.py @@ -2,7 +2,7 @@ """Third party Django app""" from django.contrib import admin """Import Django app""" -from .models import Junior +from .models import Junior, JuniorPoints # Register your models here. @admin.register(Junior) class JuniorAdmin(admin.ModelAdmin): @@ -12,3 +12,12 @@ class JuniorAdmin(admin.ModelAdmin): def __str__(self): """Return email id""" return self.auth__email + +@admin.register(JuniorPoints) +class JuniorPointsAdmin(admin.ModelAdmin): + """Junior Points Admin""" + list_display = ['junior', 'total_task_points'] + + def __str__(self): + """Return email id""" + return self.junior.auth.email diff --git a/junior/migrations/0008_juniorpoints.py b/junior/migrations/0008_juniorpoints.py new file mode 100644 index 0000000..d16b7e2 --- /dev/null +++ b/junior/migrations/0008_juniorpoints.py @@ -0,0 +1,28 @@ +# Generated by Django 4.2.2 on 2023-07-09 12:40 + +from django.db import migrations, models +import django.db.models.deletion + + +class Migration(migrations.Migration): + + dependencies = [ + ('junior', '0007_alter_junior_image'), + ] + + operations = [ + migrations.CreateModel( + name='JuniorPoints', + fields=[ + ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('total_task_points', models.IntegerField(blank=True, default=0, null=True)), + ('created_at', models.DateTimeField(auto_now_add=True)), + ('updated_at', models.DateTimeField(auto_now=True)), + ('junior', models.OneToOneField(on_delete=django.db.models.deletion.CASCADE, related_name='junior_points', to='junior.junior')), + ], + options={ + 'verbose_name': 'Junior Task Points', + 'db_table': 'junior_task_points', + }, + ), + ] diff --git a/junior/models.py b/junior/models.py index 2534b50..167983f 100644 --- a/junior/models.py +++ b/junior/models.py @@ -41,3 +41,21 @@ class Junior(models.Model): def __str__(self): """Return email id""" return f'{self.auth}' + +class JuniorPoints(models.Model): + """Junior model""" + junior = models.OneToOneField(Junior, on_delete=models.CASCADE, related_name='junior_points') + """Contact details""" + total_task_points = models.IntegerField(blank=True, null=True, default=0) + """Profile created and updated time""" + created_at = models.DateTimeField(auto_now_add=True) + updated_at = models.DateTimeField(auto_now=True) + + class Meta(object): + """ Meta class """ + db_table = 'junior_task_points' + verbose_name = 'Junior Task Points' + + def __str__(self): + """Return email id""" + return f'{self.junior.auth}'