From ab3a57b7dd8926b4733e562d8cb147ff353f70d9 Mon Sep 17 00:00:00 2001 From: Sander Roosendaal Date: Fri, 12 Jun 2020 14:14:59 +0200 Subject: [PATCH 1/3] send email when course fails --- rowers/plannedsessions.py | 6 +- rowers/tasks.py | 75 ++++++++++++++++++++++- rowers/templates/trajectoryfailemail.html | 22 +++++++ rowers/views/planviews.py | 4 +- 4 files changed, 102 insertions(+), 5 deletions(-) create mode 100644 rowers/templates/trajectoryfailemail.html diff --git a/rowers/plannedsessions.py b/rowers/plannedsessions.py index 02df37f3..d8936ca3 100644 --- a/rowers/plannedsessions.py +++ b/rowers/plannedsessions.py @@ -381,6 +381,7 @@ def add_workouts_plannedsession(ws,ps,r): record.save() job = myqueue(queue,handle_check_race_course,w.csvfilename, w.id,ps.course.id,record.id, + w.user.user.email,w.user.user.first_name, mode='coursetest') else: errors.append('Workout %i did not match session dates' % w.id) @@ -659,6 +660,7 @@ def is_session_complete_ws(ws,ps): record.save() job = myqueue(queue,handle_check_race_course,ws[0].csvfilename, ws[0].id,ps.course.id,record.id, + ws[0].user.user.email,ws[0].user.user.first_name, mode='coursetest') return (0,'not done',None) @@ -1575,7 +1577,9 @@ def add_workout_race(ws,race,r,splitsecond=0,recordid=0): comments.append('Workouts submitted to virtual events have to be public. We have changed the workout to a public workout.') job = myqueue(queue,handle_check_race_course,ws[0].csvfilename, - ws[0].id,race.course.id,record.id,splitsecond=splitsecond, + ws[0].id,race.course.id,record.id, + ws[0].user.user.email,ws[0].user.user.first_name, + splitsecond=splitsecond, referencespeed=record.referencespeed,coursedistance=race.course.distance ) diff --git a/rowers/tasks.py b/rowers/tasks.py index fcd7bc62..68a33e89 100644 --- a/rowers/tasks.py +++ b/rowers/tasks.py @@ -119,7 +119,11 @@ siteurl = SITE_URL # testing task from rowers.emails import send_template_email -from rowers.courseutils import coursetime_paths, coursetime_first, time_in_path +from rowers.courseutils import ( + coursetime_paths, coursetime_first, time_in_path, + InvalidTrajectoryError + ) + @app.task def add(x, y): @@ -342,7 +346,8 @@ def polygon_to_path(polygon,debug=True): @app.task(bind=True) def handle_check_race_course(self, f1,workoutid,courseid, - recordid,**kwargs): + recordid,useremail,userfirstname, + **kwargs): if 'debug' in kwargs: debug = kwargs['debug'] @@ -432,7 +437,15 @@ def handle_check_race_course(self, paths.append(path) # check how many times went through start polygon - entrytimes,entrydistances = time_in_path(rowdata,paths[0],maxmin='max',getall=True) + try: + entrytimes,entrydistances = time_in_path(rowdata,paths[0],maxmin='max',getall=True) + except InvalidTrajectoryError: + entrytimes = [] + entrydistances = [] + coursecompleted = False + coursemeters = 0 + coursetimeseconds = 0 + cseconds = [] cmeters = [] @@ -528,6 +541,39 @@ def handle_check_race_course(self, return 1 else: + query = 'UPDATE rowers_virtualraceresult SET coursecompleted = 0, duration = "{duration}", distance = {distance}, workoutid = {workoutid}, startsecond = {startsecond}, endsecond = {endsecond}, points={points} WHERE id={recordid}'.format( + recordid=recordid, + duration=totaltime_sec_to_string(0), + distance=0), + points=0, + workoutid=workoutid, + startsecond=startsecond, + endsecond=endsecond, + ) + + if mode == 'coursetest': + query = 'UPDATE rowers_coursetestresult SET coursecompleted = 0, duration = "{duration}", distance = {distance}, workoutid = {workoutid}, startsecond = {startsecond}, endsecond = {endsecond}, points={points} WHERE id={recordid}'.format( + recordid=recordid, + duration=totaltime_sec_to_string(0), + distance=0, + points=0, + workoutid=workoutid, + startsecond=startsecond, + endsecond=endsecond, + ) + + + with engine.connect() as conn, conn.begin(): + result = conn.execute(query) + + conn.close() + engine.dispose() + + # send email + handle_send_email_coursefail( + useremail,userfirstname, + ) + return 2 return 0 @@ -1118,6 +1164,29 @@ def handle_sendemail_raceregistration( return 1 +def handle_sendemail_coursefail( + useremail, username, **kwargs): + + if 'debug' in kwargs: + debug = kwargs['debug'] + else: + debug = True + + subject = "The validation of your course has failed" + + from_email = 'Rowsandall ' + + d = { + 'username':username, + } + + res = send_template_email(from_email,[useremail], + subject, + 'trajectoryfailemail.html', + d,**kwargs) + + return 1 + @app.task def handle_sendemail_optout( useremail, username, registeredname, racename, raceid, **kwargs): diff --git a/rowers/templates/trajectoryfailemail.html b/rowers/templates/trajectoryfailemail.html new file mode 100644 index 00000000..3022682b --- /dev/null +++ b/rowers/templates/trajectoryfailemail.html @@ -0,0 +1,22 @@ +{% extends "emailbase.html" %} +{% block body %} +

Dear {{ username }},

+ +

+ Unfortunately, the course you took did not go through all gates for + the virtual challenge. + You can check your course versus the gates by clicking on Details + in your challenge result. + Of course, you can always submit a new row! +

+ +

+ If you feel that your course should have been valid, or in case + you have questions, please + contact me by reply to this email. +

+ +

+ Best Regards, the Rowsandall Team +

+{% endblock %} diff --git a/rowers/views/planviews.py b/rowers/views/planviews.py index f596d83a..f30b51b9 100644 --- a/rowers/views/planviews.py +++ b/rowers/views/planviews.py @@ -1956,7 +1956,9 @@ def plannedsession_view(request,id=0,userid=0): record.save() job = myqueue(queue,handle_check_race_course, w.csvfilename,w.id,ps.course.id, - record.id,mode='coursetest') + record.id, + w.user.user.email,w.user.user.first_name, + mode='coursetest') intsecs = 0 microsecs = 0 From 5bb941ce88a9c4eb702e328cc275fd821beae122 Mon Sep 17 00:00:00 2001 From: Sander Roosendaal Date: Fri, 12 Jun 2020 14:26:12 +0200 Subject: [PATCH 2/3] adding ranking --- rowers/templates/virtualevent.html | 28 ++++++++++++++++++++++- rowers/templates/virtualeventranking.html | 28 ++++++++++++++++++++++- rowers/views/racesviews.py | 11 +++++++++ 3 files changed, 65 insertions(+), 2 deletions(-) diff --git a/rowers/templates/virtualevent.html b/rowers/templates/virtualevent.html index 90685498..cd3ffd22 100644 --- a/rowers/templates/virtualevent.html +++ b/rowers/templates/virtualevent.html @@ -267,7 +267,7 @@

Results

{% endif %}

- {% if results or dns %} + {% if results or dns or dnf %}

@@ -349,6 +349,32 @@ {% endfor %} + {% for result in dnf %} + + + + + + + + + + {% if race.sessiontype == 'race' %} + + {% endif %} + + + + {% endfor %} {% for result in dns %} diff --git a/rowers/templates/virtualeventranking.html b/rowers/templates/virtualeventranking.html index 42bb6299..9053c269 100644 --- a/rowers/templates/virtualeventranking.html +++ b/rowers/templates/virtualeventranking.html @@ -48,7 +48,7 @@

Results

{% endif %}

- {% if results or dns %} + {% if results or dns or dnf %}

 {{ result.username }}{{ result.teamname }}{{ result.age }}{{ result.sex }}{{ result.weightcategory }} + {% if result.adaptiveclass == 'None' %} +   + {% else %} + {{ result.adaptiveclass }} + {% endif %} + {{ result.boatclass }}{{ result.boattype }}DNF + + Details +
 
@@ -108,6 +108,32 @@ {% endfor %} + {% for result in dnf %} + + + + + + + + + + {% if race.sessiontype == 'race' %} + + {% endif %} + + + + {% endfor %} {% for result in dns %} diff --git a/rowers/views/racesviews.py b/rowers/views/racesviews.py index 1f6f4a33..9f918437 100644 --- a/rowers/views/racesviews.py +++ b/rowers/views/racesviews.py @@ -1072,6 +1072,12 @@ def virtualevent_view(request,id=0): workoutid__isnull=True, ) + dnf = resultobj.objects.filter( + race=race, + workoutid__isnull=False, + coursecompleted=False, + ) + if not request.user.is_anonymous: if race_can_register(r,race): @@ -1143,6 +1149,7 @@ def virtualevent_view(request,id=0): adaptiveclass__in=adaptiveclass, age__gte=age_min, age__lte=age_max, + coursecompleted=True, ).order_by("duration") else: results = resultobj.objects.filter( @@ -1154,6 +1161,7 @@ def virtualevent_view(request,id=0): adaptiveclass__in=adaptiveclass, age__gte=age_min, age__lte=age_max, + coursecompleted=True, ).order_by("duration","-distance") if entrycategory is not None: @@ -1179,6 +1187,8 @@ def virtualevent_view(request,id=0): coursecompleted=True, ).order_by("duration","-distance") + + if results: form = RaceResultFilterForm(records=records) else: @@ -1221,6 +1231,7 @@ def virtualevent_view(request,id=0): 'results':results, 'buttons':buttons, 'dns':dns, + 'dnf':dnf, 'records':records, 'racelogo':racelogo, 'form':form, From d9405b428102402ad1450f47126c8c2cfb8f90d8 Mon Sep 17 00:00:00 2001 From: Sander Roosendaal Date: Fri, 12 Jun 2020 14:27:24 +0200 Subject: [PATCH 3/3] adding ranking --- rowers/views/racesviews.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/rowers/views/racesviews.py b/rowers/views/racesviews.py index 9f918437..7d2e2dcf 100644 --- a/rowers/views/racesviews.py +++ b/rowers/views/racesviews.py @@ -1315,6 +1315,12 @@ def virtualevent_ranking_view(request,id=0): workoutid__isnull=True, ) + dnf = resultobj.objects.filter( + race=race, + workoutid__isnull=False, + coursecompleted=False, + ) + if not request.user.is_anonymous: if race_can_register(r,race):
 {{ result.username }}{{ result.teamname }}{{ result.age }}{{ result.sex }}{{ result.weightcategory }} + {% if result.adaptiveclass == 'None' %} +   + {% else %} + {{ result.adaptiveclass }} + {% endif %} + {{ result.boatclass }}{{ result.boattype }} + + Details + DNF