Merge pull request #25 from akirachix/feature/fixes

feat: Update user profile image functionality

Author: jmwai

.gitignore

@@ -4,4 +4,5 @@ venv/
 env/
 __pycache__/
 *.pyc
-staticfiles/
+staticfiles/
+media/profile-picture/

api/serializers.py

@@ -69,7 +69,6 @@ def create(self, validated_data):
             )
         return user
 
-
     def update(self, instance, validated_data):
         password = validated_data.pop("password", None)
         if password:
@@ -78,13 +77,12 @@ def update(self, instance, validated_data):
         device_id_str = validated_data.pop('device_id', None)
         if device_id_str:
             instance.device_id = MCU.objects.get(device_id=device_id_str)
-        elif device_id_str is not None:   
-            instance.device_id = None
-        elif 'device_id' not in self.initial_data:
-             pass
-        else:
+        elif 'device_id' in validated_data and validated_data['device_id'] is None:
             instance.device_id = None
-            
+
+        if 'image' in validated_data:
+            instance.image = validated_data['image']
+
         for attr, value in validated_data.items():
             setattr(instance, attr, value)
 

api/urls.py

@@ -26,7 +26,5 @@
     path('schema/swagger-ui/', SpectacularSwaggerView.as_view(url_name='schema'), name='swagger-ui'),
     path('schema/redoc/', SpectacularRedocView.as_view(url_name='schema'), name='redoc'),
 
-    
-   
 ]
 

api/views.py

@@ -1,7 +1,7 @@
 from rest_framework.views import APIView
 from rest_framework import viewsets
 from rest_framework.response import Response
-from rest_framework import generics, status
+from rest_framework import generics, status, parsers
 from rest_framework.reverse import reverse
 from rest_framework.permissions import AllowAny
 from rest_framework_simplejwt.tokens import AccessToken
@@ -26,8 +26,7 @@ class UserViewSet(viewsets.ModelViewSet):
     queryset = User.objects.all()
     serializer_class = UserSerializer
     permission_classes = [AllowAny]
-    
-
+    parser_classes = (parsers.MultiPartParser, parsers.FormParser, parsers.JSONParser)
 class ThresholdViewSet(viewsets.ViewSet):
     def list(self, request):
         mcus = MCU.objects.all()
@@ -76,6 +75,7 @@ class UserAPIView(generics.GenericAPIView):
     queryset = User.objects.all()
     serializer_class = UserSerializer
     permission_classes = [AllowAny]
+    parser_classes = (parsers.MultiPartParser, parsers.FormParser, parsers.JSONParser)
     def get(self, request, *args, **kwargs):
         users = self.get_queryset()
         serializer = self.get_serializer(users, many=True)
@@ -90,6 +90,7 @@ class UserDetailView(generics.RetrieveUpdateDestroyAPIView):
     queryset = User.objects.all()
     serializer_class = UserSerializer
     permission_classes = [AllowAny]
+    parser_classes = (parsers.MultiPartParser, parsers.FormParser, parsers.JSONParser)
 
 class SetPasswordView(generics.GenericAPIView):
     serializer_class = SetPasswordSerializer

kukukonnect/settings.py

@@ -23,7 +23,7 @@
 
 SECRET_KEY = os.getenv('SECRET_KEY')
 # SECURITY WARNING: don't run with debug turned on in production!
-DEBUG =False
+DEBUG=True
 ALLOWED_HOSTS = ["*"]
 # Application definition
 INSTALLED_APPS = [

users/migrations/0002_alter_user_image.py

@@ -0,0 +1,17 @@
+# Generated by Django 4.2.24 on 2025-09-22 19:04
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("users", "0001_initial"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="user",
+            name="image",
+            field=models.URLField(blank=True, null=True, verbose_name="Profile URL"),
+        ),
+    ]

users/migrations/0003_alter_user_image.py

@@ -0,0 +1,22 @@
+# Generated by Django 4.2.24 on 2025-09-23 06:41
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("users", "0002_alter_user_image"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="user",
+            name="image",
+            field=models.ImageField(
+                blank=True,
+                null=True,
+                upload_to="profile_images/",
+                verbose_name="Profile Image",
+            ),
+        ),
+    ]

users/migrations/0004_alter_user_image.py

@@ -0,0 +1,17 @@
+# Generated by Django 4.2.24 on 2025-09-23 07:44
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("users", "0003_alter_user_image"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="user",
+            name="image",
+            field=models.ImageField(blank=True, null=True, upload_to="profile_images/"),
+        ),
+    ]

users/models.py

@@ -32,7 +32,7 @@ class User(AbstractBaseUser):
     blank=True,
     verbose_name=_("Device ID")
 )
-    image = models.ImageField(_("Profile Image"), upload_to='profile_images/', blank=True, null=True)
+    image = models.ImageField(upload_to='profile_images/', blank=True, null=True)
     user_type = models.CharField(
         _("User Type"),
         max_length=20,