Merge pull request #512 from sumesh-aot/sms

Fixing issue when sms is disabled in PROD
bcgov · Feb 16, 2021 · 223c395 · 223c395
2 parents 13833aa + 163b0da
commit 223c395
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/api/app/resources/bookings/appointment/appointment_reminder_get.py b/api/app/resources/bookings/appointment/appointment_reminder_get.py
@@ -40,6 +40,7 @@ def get(self, reminder_type: str = 'email'):
         if appointments:
             for (appointment, office, timezone, user) in appointments:
                 send_reminder = False
+                user_telephone = None
                 if reminder_type == 'email':
                     if user and user.send_email_reminders:
                         send_reminder = True

diff --git a/sonar-runner/build.gradle b/sonar-runner/build.gradle
@@ -13,7 +13,7 @@ sonarqube {
         property "sonar.projectName", "Queue Management"
         property "sonar.projectKey", "org.sonarqube:queue-management"
         property "sonar.projectBaseDir", "../"
-        property "sonar.sources", "./api,./frontend/src,./appointment-frontend/src,/jobs/appointment_reminder"
+        property "sonar.sources", "./api,./frontend/src,./appointment-frontend/src,./jobs/appointment_reminder,./notifications-api"
         property "sonar.exclusions", "**/test_*.py"
         property "sonar.verbose", "true"
         property "sourceEncoding", "UTF-8"