From 33e61a3d563cbea8d4537defa4401694cc45b228 Mon Sep 17 00:00:00 2001
From: John Baldwin <jlbaldwin@gmail.com>
Date: Tue, 28 Jul 2020 13:52:25 -0400
Subject: [PATCH] A couple of inline comments for code improvement

---
 figures/models.py | 2 ++
 figures/sites.py  | 3 +++
 2 files changed, 5 insertions(+)

diff --git a/figures/models.py b/figures/models.py
index c8108e93..981e1464 100644
--- a/figures/models.py
+++ b/figures/models.py
@@ -103,6 +103,8 @@ class SiteDailyMetrics(TimeStampedModel):
     total_user_count = models.IntegerField()
     course_count = models.IntegerField()
     total_enrollment_count = models.IntegerField()
+
+    # Should change this to default value of 0
     mau = models.IntegerField(blank=True, null=True)
 
     class Meta:
diff --git a/figures/sites.py b/figures/sites.py
index dc88a473..1c2a922a 100644
--- a/figures/sites.py
+++ b/figures/sites.py
@@ -160,6 +160,9 @@ def get_courses_for_site(site):
 
 
 def get_user_ids_for_site(site):
+    """
+    We can use organization_instance.users instead
+    """
     if figures.helpers.is_multisite():
         orgs = organizations.models.Organization.objects.filter(sites__in=[site])
         mappings = organizations.models.UserOrganizationMapping.objects.filter(