42CTF/website
42CTF
/
website
13
5
Fork 10
Code Issues 10 Pull Requests 1 Projects 9 Releases Wiki Activity

fix(api): Check if user has campus before add to response #85

Merged
Danhia merged 1 commits from Starthur/website:main into main 2023-09-01 11:09:53 +02:00
Conversation 0 Commits 1 Files Changed 1 +10 -11
1 changed files with 10 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 691c53e110 - Show all commits

5
src/api/views.py
View File

@ -40,8 +40,8 @@ def bot_discord_campus(request):
all_users = UserProfileInfo.objects.select_related().order_by('-score', 'last_submission_date', 'user__username') all_users = UserProfileInfo.objects.select_related().order_by('-score', 'last_submission_date', 'user__username')
data = {} data = {}
for user in all_users: for user in all_users:
if user.discord_id: if user.campus and user.discord_id:
data[user.discord_id] = user.campus data[user.discord_id] = user.campus.name
return JsonResponse(data) return JsonResponse(data)
@ -70,4 +70,3 @@ def events_data(request, event_slug):
return JsonResponse(data) return JsonResponse(data)