A couple of inline comments for code improvement

Author: johnbaldwin

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:

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(