fix: harmonize user_id type

This commit is contained in:
Livio Bieri 2024-03-06 13:46:27 +01:00
parent 84f2761795
commit 2b92928060
2 changed files with 5 additions and 3 deletions

View File

@ -40,7 +40,7 @@ urlpatterns = [
# route to get feedback summaries for a user in a course session
# used by different roles to retrieve feedback summaries for a user
path(
"feedbacks/summaries/course-session/<signed_int:course_session_id>/user/<str:user_id>",
"feedbacks/summaries/course-session/<signed_int:course_session_id>/user/<uuid:user_id>",
get_course_session_user_feedback_summaries,
name="get_course_session_user_feedback_summaries",
),

View File

@ -1,3 +1,5 @@
import uuid
import structlog
from django.shortcuts import get_object_or_404
from rest_framework.decorators import api_view, permission_classes
@ -93,7 +95,7 @@ def get_self_evaluation_feedback_as_provider(request, learning_unit_id):
@api_view(["GET"])
@permission_classes([IsAuthenticated])
def get_course_session_user_feedback_summaries(
request, course_session_id: int, user_id: str
request, course_session_id: int, user_id: uuid.UUID
):
course_session = get_object_or_404(CourseSession, id=course_session_id)
user_to_lookup = get_object_or_404(User, id=user_id)
@ -101,7 +103,7 @@ def get_course_session_user_feedback_summaries(
if not can_view_course_completions(
user=request.user, # noqa
course_session_id=course_session_id,
target_user_id=user_id,
target_user_id=str(user_id),
):
raise PermissionDenied()